Title: [157586] trunk/Source/_javascript_Core
Revision
157586
Author
msab...@apple.com
Date
2013-10-17 11:57:41 -0700 (Thu, 17 Oct 2013)

Log Message

Remove saving callFrameRegister to JITStackFrame in JITCompiler::compileFunction()
https://bugs.webkit.org/show_bug.cgi?id=122974

Reviewed by Geoffrey Garen.

Eliminated unneeded storing to JITStackFrame.

* dfg/DFGJITCompiler.cpp:
(JSC::DFG::JITCompiler::compileFunction):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (157585 => 157586)


--- trunk/Source/_javascript_Core/ChangeLog	2013-10-17 18:52:57 UTC (rev 157585)
+++ trunk/Source/_javascript_Core/ChangeLog	2013-10-17 18:57:41 UTC (rev 157586)
@@ -1,5 +1,17 @@
 2013-10-17  Michael Saboff  <msab...@apple.com>
 
+        Remove saving callFrameRegister to JITStackFrame in JITCompiler::compileFunction()
+        https://bugs.webkit.org/show_bug.cgi?id=122974
+
+        Reviewed by Geoffrey Garen.
+
+        Eliminated unneeded storing to JITStackFrame.
+
+        * dfg/DFGJITCompiler.cpp:
+        (JSC::DFG::JITCompiler::compileFunction):
+
+2013-10-17  Michael Saboff  <msab...@apple.com>
+
         Transition cti_op_throw and cti_vm_throw to a JIT operation
         https://bugs.webkit.org/show_bug.cgi?id=122931
 

Modified: trunk/Source/_javascript_Core/dfg/DFGJITCompiler.cpp (157585 => 157586)


--- trunk/Source/_javascript_Core/dfg/DFGJITCompiler.cpp	2013-10-17 18:52:57 UTC (rev 157585)
+++ trunk/Source/_javascript_Core/dfg/DFGJITCompiler.cpp	2013-10-17 18:57:41 UTC (rev 157586)
@@ -380,8 +380,6 @@
     // we need to call out to a helper function to check whether more space is available.
     // FIXME: change this from a cti call to a DFG style operation (normal C calling conventions).
     stackCheck.link(this);
-    move(stackPointerRegister, GPRInfo::argumentGPR0);
-    poke(GPRInfo::callFrameRegister, OBJECT_OFFSETOF(struct JITStackFrame, callFrame) / sizeof(void*));
 
     emitStoreCodeOrigin(CodeOrigin(0));
     m_speculative->callOperationWithCallFrameRollbackOnException(operationStackCheck, m_codeBlock);
@@ -397,8 +395,6 @@
 
     load32(AssemblyHelpers::payloadFor((VirtualRegister)JSStack::ArgumentCount), GPRInfo::regT1);
     branch32(AboveOrEqual, GPRInfo::regT1, TrustedImm32(m_codeBlock->numParameters())).linkTo(fromArityCheck, this);
-    move(stackPointerRegister, GPRInfo::argumentGPR0);
-    poke(GPRInfo::callFrameRegister, OBJECT_OFFSETOF(struct JITStackFrame, callFrame) / sizeof(void*));
     emitStoreCodeOrigin(CodeOrigin(0));
     m_speculative->callOperationWithCallFrameRollbackOnException(m_codeBlock->m_isConstructor ? operationConstructArityCheck : operationCallArityCheck, GPRInfo::regT0);
     branchTest32(Zero, GPRInfo::regT0).linkTo(fromArityCheck, this);
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to