Title: [100878] trunk/Source/_javascript_Core
Revision
100878
Author
[email protected]
Date
2011-11-20 19:32:02 -0800 (Sun, 20 Nov 2011)

Log Message

DFG 32_64 should directly store double virtual registers on SetLocal
https://bugs.webkit.org/show_bug.cgi?id=72845

Reviewed by Oliver Hunt.
        
2% win on Kraken.

* dfg/DFGSpeculativeJIT32_64.cpp:
(JSC::DFG::SpeculativeJIT::compile):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (100877 => 100878)


--- trunk/Source/_javascript_Core/ChangeLog	2011-11-21 02:26:16 UTC (rev 100877)
+++ trunk/Source/_javascript_Core/ChangeLog	2011-11-21 03:32:02 UTC (rev 100878)
@@ -1,3 +1,15 @@
+2011-11-20  Filip Pizlo  <[email protected]>
+
+        DFG 32_64 should directly store double virtual registers on SetLocal
+        https://bugs.webkit.org/show_bug.cgi?id=72845
+
+        Reviewed by Oliver Hunt.
+        
+        2% win on Kraken.
+
+        * dfg/DFGSpeculativeJIT32_64.cpp:
+        (JSC::DFG::SpeculativeJIT::compile):
+
 2011-11-20  Noel Gordon  <[email protected]>
 
         [chromium] Remove DFG::JITCodeGenerator from the gyp projects

Modified: trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT32_64.cpp (100877 => 100878)


--- trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT32_64.cpp	2011-11-21 02:26:16 UTC (rev 100877)
+++ trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT32_64.cpp	2011-11-21 03:32:02 UTC (rev 100878)
@@ -2142,7 +2142,11 @@
         m_codeOriginForOSR = nextNode.codeOrigin;
         
         PredictedType predictedType = node.variableAccessData()->prediction();
-        if (isInt32Prediction(predictedType)) {
+        if (m_generationInfo[at(node.child1()).virtualRegister()].registerFormat() == DataFormatDouble) {
+            DoubleOperand value(this, node.child1());
+            m_jit.storeDouble(value.fpr(), JITCompiler::addressFor(node.local()));
+            noResult(m_compileIndex);
+        } else if (isInt32Prediction(predictedType)) {
             SpeculateIntegerOperand value(this, node.child1());
             m_jit.store32(value.gpr(), JITCompiler::payloadFor(node.local()));
             noResult(m_compileIndex);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to