Title: [183251] trunk/Source/_javascript_Core
- Revision
- 183251
- Author
- [email protected]
- Date
- 2015-04-23 23:36:14 -0700 (Thu, 23 Apr 2015)
Log Message
Remove a useless branch on DFGGraph::addShouldSpeculateMachineInt()
https://bugs.webkit.org/show_bug.cgi?id=144118
Patch by Benjamin Poulain <[email protected]> on 2015-04-23
Reviewed by Geoffrey Garen.
* dfg/DFGGraph.h:
(JSC::DFG::Graph::addShouldSpeculateMachineInt):
Both block do the same thing.
Modified Paths
Diff
Modified: trunk/Source/_javascript_Core/ChangeLog (183250 => 183251)
--- trunk/Source/_javascript_Core/ChangeLog 2015-04-24 06:30:16 UTC (rev 183250)
+++ trunk/Source/_javascript_Core/ChangeLog 2015-04-24 06:36:14 UTC (rev 183251)
@@ -1,3 +1,14 @@
+2015-04-23 Benjamin Poulain <[email protected]>
+
+ Remove a useless branch on DFGGraph::addShouldSpeculateMachineInt()
+ https://bugs.webkit.org/show_bug.cgi?id=144118
+
+ Reviewed by Geoffrey Garen.
+
+ * dfg/DFGGraph.h:
+ (JSC::DFG::Graph::addShouldSpeculateMachineInt):
+ Both block do the same thing.
+
2015-04-23 Joseph Pecoraro <[email protected]>
Web Inspector: Speculative fix for non-main thread auto-attach failures
Modified: trunk/Source/_javascript_Core/dfg/DFGGraph.h (183250 => 183251)
--- trunk/Source/_javascript_Core/dfg/DFGGraph.h 2015-04-24 06:30:16 UTC (rev 183250)
+++ trunk/Source/_javascript_Core/dfg/DFGGraph.h 2015-04-24 06:36:14 UTC (rev 183251)
@@ -268,12 +268,7 @@
Node* left = add->child1().node();
Node* right = add->child2().node();
- bool speculation;
- if (add->op() == ValueAdd)
- speculation = Node::shouldSpeculateMachineInt(left, right);
- else
- speculation = Node::shouldSpeculateMachineInt(left, right);
-
+ bool speculation = Node::shouldSpeculateMachineInt(left, right);
return speculation && !hasExitSite(add, Int52Overflow);
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
