Title: [199796] trunk/Source/_javascript_Core
- Revision
- 199796
- Author
- [email protected]
- Date
- 2016-04-20 16:28:29 -0700 (Wed, 20 Apr 2016)
Log Message
[JSC] DFG should not generate two jumps when the target of DoubleBranch is the next block
https://bugs.webkit.org/show_bug.cgi?id=156815
Patch by Benjamin Poulain <[email protected]> on 2016-04-20
Reviewed by Mark Lam.
* dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculativeJIT::compilePeepHoleDoubleBranch):
Modified Paths
Diff
Modified: trunk/Source/_javascript_Core/ChangeLog (199795 => 199796)
--- trunk/Source/_javascript_Core/ChangeLog 2016-04-20 22:56:40 UTC (rev 199795)
+++ trunk/Source/_javascript_Core/ChangeLog 2016-04-20 23:28:29 UTC (rev 199796)
@@ -1,5 +1,15 @@
2016-04-20 Benjamin Poulain <[email protected]>
+ [JSC] DFG should not generate two jumps when the target of DoubleBranch is the next block
+ https://bugs.webkit.org/show_bug.cgi?id=156815
+
+ Reviewed by Mark Lam.
+
+ * dfg/DFGSpeculativeJIT.cpp:
+ (JSC::DFG::SpeculativeJIT::compilePeepHoleDoubleBranch):
+
+2016-04-20 Benjamin Poulain <[email protected]>
+
[JSC] Add register reuse for ArithAdd of an Int32 and constant in DFG
https://bugs.webkit.org/show_bug.cgi?id=155164
Modified: trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT.cpp (199795 => 199796)
--- trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT.cpp 2016-04-20 22:56:40 UTC (rev 199795)
+++ trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT.cpp 2016-04-20 23:28:29 UTC (rev 199796)
@@ -1334,7 +1334,12 @@
{
BasicBlock* taken = branchNode->branchData()->taken.block;
BasicBlock* notTaken = branchNode->branchData()->notTaken.block;
-
+
+ if (taken == nextBlock()) {
+ condition = MacroAssembler::invert(condition);
+ std::swap(taken, notTaken);
+ }
+
SpeculateDoubleOperand op1(this, node->child1());
SpeculateDoubleOperand op2(this, node->child2());
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
