Changeset 34906 in webkit for trunk/JavaScriptCore/kjs/ExecState.cpp

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/ExecState.cpp (modified) (2 diffs)

trunk/JavaScriptCore/kjs/ExecState.cpp

@@ -40 +40 @@
     , m_registerFile(0)
     , m_scopeChain(globalScopeChain)
-    , m_callFrameOffset(-1)
+    , m_callFrame(0)
 {
 }
 
-ExecState::ExecState(ExecState* exec, RegisterFile* registerFile, ScopeChainNode* scopeChain, int callFrameOffset)
+ExecState::ExecState(ExecState* exec, RegisterFile* registerFile, ScopeChainNode* scopeChain, Register* callFrame)
     : m_globalObject(exec->m_globalObject)
     , m_globalThisValue(exec->m_globalThisValue)
@@ -52 +52 @@
     , m_registerFile(registerFile)
     , m_scopeChain(scopeChain)
-    , m_callFrameOffset(callFrameOffset)
+    , m_callFrame(callFrame)
 {
     ASSERT(!exec->m_exception);