Changeset 34906 in webkit for trunk/JavaScriptCore/kjs/DebuggerCallFrame.h

Timestamp:

Jun 30, 2008, 10:45:21 PM (17 years ago)

Author:

[email protected]

Message:

2008-06-30 Geoffrey Garen <​[email protected]>

Reviewed by Oliver Hunt.

Removed the "registerBase" abstraction. Since the register file never
reallocates, we can keep direct pointers into it, instead of
<registerBase, offset> tuples.

SunSpider says 0.8% faster.

File:

• trunk/JavaScriptCore/kjs/DebuggerCallFrame.h (modified) (3 diffs)

trunk/JavaScriptCore/kjs/DebuggerCallFrame.h

@@ -48 +48 @@
         } Type;
 
-        DebuggerCallFrame(JSGlobalObject* dynamicGlobalObject, const CodeBlock* codeBlock, ScopeChainNode* scopeChain, JSValue* exception, Register** registerBase, int registerOffset)
+        DebuggerCallFrame(JSGlobalObject* dynamicGlobalObject, const CodeBlock* codeBlock, ScopeChainNode* scopeChain, Register* r, JSValue* exception)
             : m_dynamicGlobalObject(dynamicGlobalObject)
             , m_codeBlock(codeBlock)
             , m_scopeChain(scopeChain)
+            , m_registers(r)
             , m_exception(exception)
-            , m_registerBase(registerBase)
-            , m_registerOffset(registerOffset)
         {
         }
@@ -67 +66 @@
 
     private:
-        Register* r() const;
         Register* callFrame() const;
 
@@ -73 +71 @@
         const CodeBlock* m_codeBlock;
         ScopeChainNode* m_scopeChain;
+        Register* m_registers;
         JSValue* m_exception;
-        Register** m_registerBase;
-        int m_registerOffset;
     };