Title: [102629] trunk/Source/_javascript_Core
Revision
102629
Author
[email protected]
Date
2011-12-12 15:08:35 -0800 (Mon, 12 Dec 2011)

Log Message

ARMv7 version of DFG soft modulo does register allocation inside of control flow
https://bugs.webkit.org/show_bug.cgi?id=74354

Reviewed by Gavin Barraclough.

* dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculativeJIT::compileSoftModulo):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (102628 => 102629)


--- trunk/Source/_javascript_Core/ChangeLog	2011-12-12 23:05:34 UTC (rev 102628)
+++ trunk/Source/_javascript_Core/ChangeLog	2011-12-12 23:08:35 UTC (rev 102629)
@@ -1,3 +1,13 @@
+2011-12-12  Filip Pizlo  <[email protected]>
+
+        ARMv7 version of DFG soft modulo does register allocation inside of control flow
+        https://bugs.webkit.org/show_bug.cgi?id=74354
+
+        Reviewed by Gavin Barraclough.
+
+        * dfg/DFGSpeculativeJIT.cpp:
+        (JSC::DFG::SpeculativeJIT::compileSoftModulo):
+
 2011-12-12  Andy Wingo  <[email protected]>
 
         Simplify autotools configure.ac

Modified: trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT.cpp (102628 => 102629)


--- trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT.cpp	2011-12-12 23:05:34 UTC (rev 102628)
+++ trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT.cpp	2011-12-12 23:08:35 UTC (rev 102629)
@@ -1980,6 +1980,8 @@
 #else
     GPRTemporary result(this);
     GPRTemporary scratch(this);
+    GPRTemporary scratch3(this);
+    GPRReg scratchGPR3 = scratch3.gpr();
     GPRReg resultGPR = result.gpr();
     GPRReg scratchGPR = scratch.gpr();
 #endif
@@ -2029,8 +2031,6 @@
     m_jit.assembler().cdq();
     m_jit.assembler().idivl_r(scratchGPR2);
 #elif CPU(ARM_THUMB2)
-    GPRTemporary scratch3(this);
-    GPRReg scratchGPR3 = scratch3.gpr();
     m_jit.countLeadingZeros32(scratchGPR2, scratchGPR);
     m_jit.countLeadingZeros32(resultGPR, scratchGPR3);
     m_jit.sub32(scratchGPR3, scratchGPR);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to