Changeset 38688 in webkit for trunk/JavaScriptCore/bytecode/Instruction.h

Timestamp:

Nov 21, 2008, 7:34:43 PM (17 years ago)

Author:

[email protected]

Message:

2008-11-21 Gavin Barraclough <​[email protected]>

Reviewed by Oliver Hunt.

Add (really) polymorphic caching for get by id self.
Very similar to caching of prototype accesses, described below.

Oh, also, probably shouldn't have been leaking those structure list objects.

4% preogression on deltablue.

• bytecode/CodeBlock.cpp: (JSC::CodeBlock::dump): (JSC::CodeBlock::derefStructures): (JSC::PrototypeStructureList::derefStructures):
• bytecode/Instruction.h:
• bytecode/Opcode.h:
• interpreter/Interpreter.cpp: (JSC::Interpreter::privateExecute): (JSC::Interpreter::cti_op_get_by_id_self_fail):
• jit/JIT.cpp: (JSC::JIT::privateCompileMainPass): (JSC::JIT::privateCompileGetByIdSelfList): (JSC::JIT::patchGetByIdSelf):
• jit/JIT.h: (JSC::JIT::compileGetByIdSelfList):

File:

• trunk/JavaScriptCore/bytecode/Instruction.h (modified) (5 diffs)

trunk/JavaScriptCore/bytecode/Instruction.h

@@ -34 +34 @@
 #include <wtf/VectorTraits.h>
 
-#define PROTOTYPE_LIST_CACHE_SIZE 4
+#define POLYMORPHIC_LIST_CACHE_SIZE 4
 
 namespace JSC {
@@ -43 +43 @@
 
     // Structure used by op_get_by_id_proto_list instruction to hold data off the main opcode stream.
-    struct PrototypeStructureList {
-        struct ProtoStubInfo {
+    struct PolymorphicAccessStructureList {
+        struct PolymorphicStubInfo {
             Structure* base;
             Structure* proto;
@@ -57 +57 @@
                 stubRoutine = _stubRoutine;
             }
-        } list[PROTOTYPE_LIST_CACHE_SIZE];
+        } list[POLYMORPHIC_LIST_CACHE_SIZE];
         
-        PrototypeStructureList(Structure* firstBase, Structure* firstProto, int cachedOffset, void* stubRoutine)
+        PolymorphicAccessStructureList(Structure* firstBase, Structure* firstProto, int cachedOffset, void* stubRoutine)
         {
             list[0].set(firstBase, firstProto, cachedOffset, stubRoutine);
+        }
+
+        void derefStructures(int count)
+        {
+            for (int i = 0; i < count; ++i) {
+                PolymorphicStubInfo& info = list[i];
+
+                ASSERT(info.base);
+                info.base->deref();
+
+                if (info.proto)
+                    info.proto->deref();
+
+                if (info.stubRoutine)
+                    WTF::fastFreeExecutable(info.stubRoutine);
+            }
         }
     };
@@ -78 +94 @@
         Instruction(StructureChain* structureChain) { u.structureChain = structureChain; }
         Instruction(JSCell* jsCell) { u.jsCell = jsCell; }
-        Instruction(PrototypeStructureList* prototypeStructure) { u.prototypeStructure = prototypeStructure; }
+        Instruction(PolymorphicAccessStructureList* polymorphicStructures) { u.polymorphicStructures = polymorphicStructures; }
 
         union {
@@ -87 +103 @@
             JSCell* jsCell;
             ResultType::Type resultType;
-            PrototypeStructureList* prototypeStructure;
+            PolymorphicAccessStructureList* polymorphicStructures;
         } u;
     };