Changeset 34906 in webkit for trunk/JavaScriptCore/kjs/DebuggerCallFrame.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/DebuggerCallFrame.cpp (modified) (2 diffs)

trunk/JavaScriptCore/kjs/DebuggerCallFrame.cpp

@@ -37 +37 @@
 namespace KJS {
 
-Register* DebuggerCallFrame::r() const
-{
-    return *m_registerBase + m_registerOffset;
-}
-
 Register* DebuggerCallFrame::callFrame() const
 {
-    return r() - m_codeBlock->numLocals - RegisterFile::CallFrameHeaderSize;
+    return m_registers - m_codeBlock->numLocals - RegisterFile::CallFrameHeaderSize;
 }
 
@@ -71 +66 @@
         return 0;
 
-    return static_cast<JSObject*>(r()[m_codeBlock->thisRegister].u.jsValue);
+    return static_cast<JSObject*>(m_registers[m_codeBlock->thisRegister].u.jsValue);
 }