Title: [102169] trunk/Source/_javascript_Core
Revision
102169
Author
[email protected]
Date
2011-12-06 12:48:27 -0800 (Tue, 06 Dec 2011)

Log Message

Don't allocate register in typedarray control flow
https://bugs.webkit.org/show_bug.cgi?id=73944

Reviewed by Gavin Barraclough.

Move a temporary allocation outside of control flow.

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

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (102168 => 102169)


--- trunk/Source/_javascript_Core/ChangeLog	2011-12-06 20:47:31 UTC (rev 102168)
+++ trunk/Source/_javascript_Core/ChangeLog	2011-12-06 20:48:27 UTC (rev 102169)
@@ -1,3 +1,15 @@
+2011-12-06  Oliver Hunt  <[email protected]>
+
+        Don't allocate register in typedarray control flow
+        https://bugs.webkit.org/show_bug.cgi?id=73944
+
+        Reviewed by Gavin Barraclough.
+
+        Move a temporary allocation outside of control flow.
+
+        * dfg/DFGSpeculativeJIT.cpp:
+        (JSC::DFG::SpeculativeJIT::compileGetByValOnFloatTypedArray):
+
 2011-12-06  Gavin Barraclough  <[email protected]>
 
         https://bugs.webkit.org/show_bug.cgi?id=68328

Modified: trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT.cpp (102168 => 102169)


--- trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT.cpp	2011-12-06 20:47:31 UTC (rev 102168)
+++ trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT.cpp	2011-12-06 20:48:27 UTC (rev 102169)
@@ -1734,8 +1734,8 @@
     FPRTemporary result(this);
     FPRReg resultReg = result.fpr();
     ASSERT(speculationRequirements != NoTypedArraySpecCheck);
+    GPRTemporary scratch(this);
     MacroAssembler::Jump inBounds = m_jit.branch32(MacroAssembler::Below, propertyReg, MacroAssembler::Address(baseReg, descriptor.m_lengthOffset));
-    GPRTemporary scratch(this);
     m_jit.move(MacroAssembler::Imm32(0), scratch.gpr());
     m_jit.convertInt32ToDouble(scratch.gpr(), resultReg);
     MacroAssembler::Jump outOfBounds = m_jit.jump();
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to