Title: [112018] trunk/Source/_javascript_Core
- Revision
- 112018
- Author
- [email protected]
- Date
- 2012-03-24 19:29:25 -0700 (Sat, 24 Mar 2012)
Log Message
DFG 64-bit Branch implementation should not be creating a JSValueOperand that
it isn't going to use
https://bugs.webkit.org/show_bug.cgi?id=82136
Reviewed by Geoff Garen.
* dfg/DFGSpeculativeJIT64.cpp:
(JSC::DFG::SpeculativeJIT::emitBranch):
Modified Paths
Diff
Modified: trunk/Source/_javascript_Core/ChangeLog (112017 => 112018)
--- trunk/Source/_javascript_Core/ChangeLog 2012-03-25 01:41:33 UTC (rev 112017)
+++ trunk/Source/_javascript_Core/ChangeLog 2012-03-25 02:29:25 UTC (rev 112018)
@@ -1,3 +1,14 @@
+2012-03-24 Filip Pizlo <[email protected]>
+
+ DFG 64-bit Branch implementation should not be creating a JSValueOperand that
+ it isn't going to use
+ https://bugs.webkit.org/show_bug.cgi?id=82136
+
+ Reviewed by Geoff Garen.
+
+ * dfg/DFGSpeculativeJIT64.cpp:
+ (JSC::DFG::SpeculativeJIT::emitBranch):
+
2012-03-24 Kevin Ollivier <[email protected]>
[wx] Unreviewed. Fix the build after WTF move.
Modified: trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT64.cpp (112017 => 112018)
--- trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT64.cpp 2012-03-25 01:41:33 UTC (rev 112017)
+++ trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT64.cpp 2012-03-25 02:29:25 UTC (rev 112018)
@@ -1662,9 +1662,6 @@
void SpeculativeJIT::emitBranch(Node& node)
{
- JSValueOperand value(this, node.child1());
- GPRReg valueGPR = value.gpr();
-
BlockIndex taken = node.takenBlockIndex();
BlockIndex notTaken = node.notTakenBlockIndex();
@@ -1695,6 +1692,9 @@
noResult(m_compileIndex);
} else {
+ JSValueOperand value(this, node.child1());
+ GPRReg valueGPR = value.gpr();
+
bool predictBoolean = isBooleanPrediction(m_jit.getPrediction(node.child1()));
if (predictBoolean) {
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
