Title: [158857] trunk/Source/_javascript_Core
Revision
158857
Author
[email protected]
Date
2013-11-07 11:23:12 -0800 (Thu, 07 Nov 2013)

Log Message

Remove unneeded moving of ESP to ECX in callToJavaScript for COMPILER(MSVC)
https://bugs.webkit.org/show_bug.cgi?id=123998

Reviewed by Mark Lam.

Dead code removal.  Passing esp as the first "C" argument to a _javascript_
function is no longer needed.

* jit/ThunkGenerators.cpp:
(JSC::callToJavaScript):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (158856 => 158857)


--- trunk/Source/_javascript_Core/ChangeLog	2013-11-07 19:04:53 UTC (rev 158856)
+++ trunk/Source/_javascript_Core/ChangeLog	2013-11-07 19:23:12 UTC (rev 158857)
@@ -1,3 +1,16 @@
+2013-11-07  Michael Saboff  <[email protected]>
+
+        Remove unneeded moving of ESP to ECX in callToJavaScript for COMPILER(MSVC)
+        https://bugs.webkit.org/show_bug.cgi?id=123998
+
+        Reviewed by Mark Lam.
+
+        Dead code removal.  Passing esp as the first "C" argument to a _javascript_
+        function is no longer needed.
+
+        * jit/ThunkGenerators.cpp:
+        (JSC::callToJavaScript):
+
 2013-11-07  Julien Brianceau  <[email protected]>
 
         Fix build for architectures with 4 argument registers (broken since r158820).

Modified: trunk/Source/_javascript_Core/jit/ThunkGenerators.cpp (158856 => 158857)


--- trunk/Source/_javascript_Core/jit/ThunkGenerators.cpp	2013-11-07 19:04:53 UTC (rev 158856)
+++ trunk/Source/_javascript_Core/jit/ThunkGenerators.cpp	2013-11-07 19:23:12 UTC (rev 158857)
@@ -206,9 +206,6 @@
     jit.push(X86Registers::ebx);
 
     jit.subPtr(CCallHelpers::TrustedImm32(EXTRA_STACK_SIZE), X86Registers::esp);
-#if COMPILER(MSVC)
-    jit.move(X86Registers::esp, X86Registers::ecx);
-#endif
 
     jit.load32(CCallHelpers::Address(X86Registers::esp, EXTRA_STACK_SIZE + 24), GPRInfo::callFrameRegister);
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to