Changeset 36802 in webkit for trunk/JavaScriptCore/kjs

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):

Location:

trunk/JavaScriptCore/kjs

Files:

• JSImmediate.h (modified) (1 diff)
• TypeInfo.h (modified) (2 diffs)

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)
         {

trunk/JavaScriptCore/kjs/TypeInfo.h

@@ -33 +33 @@
 
     // WebCore uses this to make document.all and style.filter undetectable.
-    static const unsigned MasqueradesAsUndefined = 0x1;
-    static const unsigned ImplementsHasInstance = 0x2;
+    static const unsigned MasqueradesAsUndefined = 1;
+    static const unsigned ImplementsHasInstance = 1 << 1;
+    static const unsigned OverridesHasInstance = 1 << 2;
 
     class TypeInfo {
@@ -45 +46 @@
         bool masqueradesAsUndefined() const { return m_flags & MasqueradesAsUndefined; }
         bool implementsHasInstance() const { return m_flags & ImplementsHasInstance; }
+        bool overridesHasInstance() const { return m_flags & OverridesHasInstance; }
 
+        unsigned flags() const { return m_flags; }
     private:
         JSType m_type;