Title: [98832] trunk/Source/_javascript_Core
- Revision
- 98832
- Author
- [email protected]
- Date
- 2011-10-30 17:21:15 -0700 (Sun, 30 Oct 2011)
Log Message
DFG ValueAdd(string, int) should not fail speculation
https://bugs.webkit.org/show_bug.cgi?id=71195
Reviewed by Geoff Garen.
1% speed-up on V8.
* dfg/DFGNode.h:
(JSC::DFG::Node::shouldNotSpeculateInteger):
(JSC::DFG::Node::shouldSpeculateInteger):
Modified Paths
Diff
Modified: trunk/Source/_javascript_Core/ChangeLog (98831 => 98832)
--- trunk/Source/_javascript_Core/ChangeLog 2011-10-30 22:47:31 UTC (rev 98831)
+++ trunk/Source/_javascript_Core/ChangeLog 2011-10-31 00:21:15 UTC (rev 98832)
@@ -1,5 +1,18 @@
2011-10-30 Filip Pizlo <[email protected]>
+ DFG ValueAdd(string, int) should not fail speculation
+ https://bugs.webkit.org/show_bug.cgi?id=71195
+
+ Reviewed by Geoff Garen.
+
+ 1% speed-up on V8.
+
+ * dfg/DFGNode.h:
+ (JSC::DFG::Node::shouldNotSpeculateInteger):
+ (JSC::DFG::Node::shouldSpeculateInteger):
+
+2011-10-30 Filip Pizlo <[email protected]>
+
The DFG inliner should not flush the callee
https://bugs.webkit.org/show_bug.cgi?id=71191
Modified: trunk/Source/_javascript_Core/dfg/DFGNode.h (98831 => 98832)
--- trunk/Source/_javascript_Core/dfg/DFGNode.h 2011-10-30 22:47:31 UTC (rev 98831)
+++ trunk/Source/_javascript_Core/dfg/DFGNode.h 2011-10-31 00:21:15 UTC (rev 98832)
@@ -967,7 +967,7 @@
static bool shouldSpeculateInteger(Node& op1, Node& op2)
{
- return !(op1.shouldNotSpeculateInteger() || op2.shouldNotSpeculateInteger()) && (op1.shouldSpeculateInteger() || op2.shouldSpeculateInteger());
+ return op1.shouldSpeculateInteger() && op2.shouldSpeculateInteger();
}
static bool shouldSpeculateNumber(Node& op1, Node& op2)
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
