Title: [102707] trunk/Source/_javascript_Core
Revision
102707
Author
fpi...@apple.com
Date
2011-12-13 14:12:33 -0800 (Tue, 13 Dec 2011)

Log Message

DFG GetByVal CSE rule should match PutByValAlias
https://bugs.webkit.org/show_bug.cgi?id=74390

Reviewed by Geoff Garen.
        
Tiny win on some benchmarks. Maybe a 0.2% win on SunSpider.

* dfg/DFGPropagator.cpp:
(JSC::DFG::Propagator::getByValLoadElimination):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (102706 => 102707)


--- trunk/Source/_javascript_Core/ChangeLog	2011-12-13 22:10:05 UTC (rev 102706)
+++ trunk/Source/_javascript_Core/ChangeLog	2011-12-13 22:12:33 UTC (rev 102707)
@@ -1,3 +1,15 @@
+2011-12-13  Filip Pizlo  <fpi...@apple.com>
+
+        DFG GetByVal CSE rule should match PutByValAlias
+        https://bugs.webkit.org/show_bug.cgi?id=74390
+
+        Reviewed by Geoff Garen.
+        
+        Tiny win on some benchmarks. Maybe a 0.2% win on SunSpider.
+
+        * dfg/DFGPropagator.cpp:
+        (JSC::DFG::Propagator::getByValLoadElimination):
+
 2011-12-13  Andy Wingo  <wi...@igalia.com>
 
         Fix interpreter debug build.

Modified: trunk/Source/_javascript_Core/dfg/DFGPropagator.cpp (102706 => 102707)


--- trunk/Source/_javascript_Core/dfg/DFGPropagator.cpp	2011-12-13 22:10:05 UTC (rev 102706)
+++ trunk/Source/_javascript_Core/dfg/DFGPropagator.cpp	2011-12-13 22:12:33 UTC (rev 102707)
@@ -1075,6 +1075,7 @@
                     return index;
                 break;
             case PutByVal:
+            case PutByValAlias:
                 if (!byValIsPure(node))
                     return NoNode;
                 if (node.child1() == child1 && canonicalize(node.child2()) == canonicalize(child2))
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to