Title: [96860] trunk/Source/_javascript_Core
Revision
96860
Author
fpi...@apple.com
Date
2011-10-06 14:43:00 -0700 (Thu, 06 Oct 2011)

Log Message

DFG implementation of UInt32ToNumber is missing a break statement
https://bugs.webkit.org/show_bug.cgi?id=69552

Reviewed by Oliver Hunt.

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

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (96859 => 96860)


--- trunk/Source/_javascript_Core/ChangeLog	2011-10-06 21:36:30 UTC (rev 96859)
+++ trunk/Source/_javascript_Core/ChangeLog	2011-10-06 21:43:00 UTC (rev 96860)
@@ -1,3 +1,15 @@
+2011-10-06  Filip Pizlo  <fpi...@apple.com>
+
+        DFG implementation of UInt32ToNumber is missing a break statement
+        https://bugs.webkit.org/show_bug.cgi?id=69552
+
+        Reviewed by Oliver Hunt.
+
+        * dfg/DFGSpeculativeJIT32_64.cpp:
+        (JSC::DFG::SpeculativeJIT::compile):
+        * dfg/DFGSpeculativeJIT64.cpp:
+        (JSC::DFG::SpeculativeJIT::compile):
+
 2011-10-06  Gavin Barraclough  <barraclo...@apple.com>
 
         Unreviewed build fix for DFG JIT 32_64 release builds.

Modified: trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT32_64.cpp (96859 => 96860)


--- trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT32_64.cpp	2011-10-06 21:36:30 UTC (rev 96859)
+++ trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT32_64.cpp	2011-10-06 21:43:00 UTC (rev 96860)
@@ -806,6 +806,7 @@
             positive.link(&m_jit);
             
             doubleResult(outputFPR, m_compileIndex);
+            break;
         }
 
         IntegerOperand op1(this, node.child1());

Modified: trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT64.cpp (96859 => 96860)


--- trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT64.cpp	2011-10-06 21:36:30 UTC (rev 96859)
+++ trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT64.cpp	2011-10-06 21:43:00 UTC (rev 96860)
@@ -924,6 +924,7 @@
             positive.link(&m_jit);
             
             doubleResult(outputFPR, m_compileIndex);
+            break;
         }
 
         IntegerOperand op1(this, node.child1());
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to