Changeset 98831 in webkit for trunk/Source/JavaScriptCore/bytecode/CodeOrigin.h

Timestamp:

Oct 30, 2011, 3:47:31 PM (14 years ago)

Author:

[email protected]

Message:

The DFG inliner should not flush the callee
​https://bugs.webkit.org/show_bug.cgi?id=71191

Reviewed by Oliver Hunt.

0.6% speed-up on V8.

• bytecode/CodeBlock.cpp:

(JSC::CodeBlock::visitAggregate):

• bytecode/CodeOrigin.h:
• dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::flush):
(JSC::DFG::ByteCodeParser::handleInlining):
(JSC::DFG::ByteCodeParser::parseBlock):
(JSC::DFG::ByteCodeParser::InlineStackEntry::InlineStackEntry):
(JSC::DFG::ByteCodeParser::parse):

• dfg/DFGJITCompiler.cpp:

(JSC::DFG::JITCompiler::exitSpeculativeWithOSR):

• dfg/DFGJITCompiler32_64.cpp:

(JSC::DFG::JITCompiler::exitSpeculativeWithOSR):

• interpreter/CallFrame.cpp:

(JSC::CallFrame::trueCallerFrameSlow):

File:

• trunk/Source/JavaScriptCore/bytecode/CodeOrigin.h (modified) (2 diffs)

trunk/Source/JavaScriptCore/bytecode/CodeOrigin.h

@@ -35 +35 @@
 struct InlineCallFrame;
 class ExecutableBase;
+class JSFunction;
 
 struct CodeOrigin {
@@ -76 +77 @@
 struct InlineCallFrame {
     WriteBarrier<ExecutableBase> executable;
+    WriteBarrier<JSFunction> callee;
+    CodeOrigin caller;
     unsigned stackOffset;
-    unsigned calleeVR;
-    CodeOrigin caller;
     unsigned numArgumentsIncludingThis : 31;
     bool isCall : 1;