Changeset 36802 in webkit for trunk/JavaScriptCore/kjs/JSImmediate.h

Timestamp:

Sep 23, 2008, 12:46:55 AM (17 years ago)

Author:

[email protected]

Message:

JavaScriptCore:

2008-09-23 Maciej Stachowiak <​[email protected]>

Reviewed by Darin.

• speed up instanceof some more ​https://bugs.webkit.org/show_bug.cgi?id=20818

~2% speedup on EarleyBoyer

The idea here is to record in the StructureID whether the class
needs a special hasInstance or if it can use the normal logic from
JSObject.

Based on this I inlined the real work directly into
cti_op_instanceof and put the fastest checks up front and the
error handling at the end (so it should be fairly straightforward
to split off the beginning to be inlined if desired).

I only did this for CTI, not the bytecode interpreter.

• API/JSCallbackObject.h: (JSC::JSCallbackObject::createStructureID):
• ChangeLog:
• VM/Machine.cpp: (JSC::Machine::cti_op_instanceof):
• kjs/JSImmediate.h: (JSC::JSImmediate::isAnyImmediate):
• kjs/TypeInfo.h: (JSC::TypeInfo::overridesHasInstance): (JSC::TypeInfo::flags):

WebCore:

2008-09-23 Maciej Stachowiak <​[email protected]>

Reviewed by Darin.

• speed up instanceof some more ​https://bugs.webkit.org/show_bug.cgi?id=20818

~2% speedup on EarleyBoyer

(WebCore updates.)

• bindings/js/JSQuarantinedObjectWrapper.h: (WebCore::JSQuarantinedObjectWrapper::createStructureID):

File:

• trunk/JavaScriptCore/kjs/JSImmediate.h (modified) (1 diff)

trunk/JavaScriptCore/kjs/JSImmediate.h

@@ -155 +155 @@
         }
 
+        static ALWAYS_INLINE bool isAnyImmediate(const JSValue* v1, const JSValue* v2, JSValue* v3)
+        {
+            return (reinterpret_cast<uintptr_t>(v1) | reinterpret_cast<uintptr_t>(v2) | reinterpret_cast<uintptr_t>(v3)) & TagMask;
+        }
+
         static ALWAYS_INLINE bool areBothImmediate(const JSValue* v1, const JSValue* v2)
         {