Changeset 58990 in webkit for trunk/JavaScriptCore/runtime

Timestamp:

May 7, 2010, 7:18:45 PM (15 years ago)

Author:

[email protected]

Message:

Optimized o[s] where o is a cell and s is a string, removing some old
code that wasn't really tuned for the JIT.

Reviewed by Darin Adler.

SunSpider says 0.8% faster.

• jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):

• runtime/JSCell.h:

File:

• trunk/JavaScriptCore/runtime/JSCell.h (modified) (2 diffs)

trunk/JavaScriptCore/runtime/JSCell.h

@@ -112 +112 @@
         void setVPtr(void* vptr) { *reinterpret_cast<void**>(this) = vptr; }
 
+        // FIXME: Rename getOwnPropertySlot to virtualGetOwnPropertySlot, and
+        // fastGetOwnPropertySlot to getOwnPropertySlot. Callers should always
+        // call this function, not its slower virtual counterpart. (For integer
+        // property names, we want a similar interface with appropriate optimizations.)
+        bool fastGetOwnPropertySlot(ExecState*, const Identifier& propertyName, PropertySlot&);
+
     protected:
         static const unsigned AnonymousSlotCount = 0;
@@ -117 +123 @@
     private:
         // Base implementation; for non-object classes implements getPropertySlot.
-        bool fastGetOwnPropertySlot(ExecState*, const Identifier& propertyName, PropertySlot&);
         virtual bool getOwnPropertySlot(ExecState*, const Identifier& propertyName, PropertySlot&);
         virtual bool getOwnPropertySlot(ExecState*, unsigned propertyName, PropertySlot&);