Title: [142780] trunk/Source/_javascript_Core
Revision
142780
Author
[email protected]
Date
2013-02-13 13:15:32 -0800 (Wed, 13 Feb 2013)

Log Message

ForwardInt32ToDouble is not in DFG::MinifiedNode's list of relevant node types
https://bugs.webkit.org/show_bug.cgi?id=109726

Reviewed by Gavin Barraclough.
        
This is asymptomatic because ForwardInt32ToDouble is only used in SetLocals, in
which case the value is already stored to the stack.  Still, we should fix this.

* dfg/DFGMinifiedNode.h:
(JSC::DFG::belongsInMinifiedGraph):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (142779 => 142780)


--- trunk/Source/_javascript_Core/ChangeLog	2013-02-13 21:00:01 UTC (rev 142779)
+++ trunk/Source/_javascript_Core/ChangeLog	2013-02-13 21:15:32 UTC (rev 142780)
@@ -1,3 +1,16 @@
+2013-02-13  Filip Pizlo  <[email protected]>
+
+        ForwardInt32ToDouble is not in DFG::MinifiedNode's list of relevant node types
+        https://bugs.webkit.org/show_bug.cgi?id=109726
+
+        Reviewed by Gavin Barraclough.
+        
+        This is asymptomatic because ForwardInt32ToDouble is only used in SetLocals, in
+        which case the value is already stored to the stack.  Still, we should fix this.
+
+        * dfg/DFGMinifiedNode.h:
+        (JSC::DFG::belongsInMinifiedGraph):
+
 2013-02-12  Filip Pizlo  <[email protected]>
 
         DFG LogicalNot/Branch peephole removal and inversion ignores the possibility of things exiting

Modified: trunk/Source/_javascript_Core/dfg/DFGMinifiedNode.h (142779 => 142780)


--- trunk/Source/_javascript_Core/dfg/DFGMinifiedNode.h	2013-02-13 21:00:01 UTC (rev 142779)
+++ trunk/Source/_javascript_Core/dfg/DFGMinifiedNode.h	2013-02-13 21:15:32 UTC (rev 142780)
@@ -45,6 +45,7 @@
     case WeakJSConstant:
     case ValueToInt32:
     case Int32ToDouble:
+    case ForwardInt32ToDouble:
     case UInt32ToNumber:
     case DoubleAsInt32:
     case PhantomArguments:
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to