Changeset 156830 in webkit

Timestamp:

Oct 3, 2013 1:43:07 AM (11 years ago)

Author:

commit-queue@webkit.org

Message:

DFG: ConstProp the pattern ValueToInt32(Bool(x)) -> Int32(x)
​https://bugs.webkit.org/show_bug.cgi?id=122263

Patch by Nadav Rotem <​nrotem@apple.com> on 2013-10-03
Reviewed by Geoffrey Garen.

• dfg/DFGAbstractInterpreterInlines.h:

(JSC::DFG::::executeEffects):

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• dfg/DFGAbstractInterpreterInlines.h (modified) (1 diff)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2013-10-03  Nadav Rotem  <nrotem@apple.com>
+
+        DFG: ConstProp the pattern ValueToInt32(Bool(x)) -> Int32(x)
+        https://bugs.webkit.org/show_bug.cgi?id=122263
+
+        Reviewed by Geoffrey Garen.
+
+        * dfg/DFGAbstractInterpreterInlines.h:
+        (JSC::DFG::::executeEffects):
+
 2013-10-02  Dan Bernstein  <mitz@apple.com>
 

trunk/Source/JavaScriptCore/dfg/DFGAbstractInterpreterInlines.h

@@ -286 +286 @@
     case ValueToInt32: {
         JSValue child = forNode(node->child1()).value();
-        if (child && child.isNumber()) {
-            if (child.isInt32())
-                setConstant(node, child);
-            else
-                setConstant(node, JSValue(JSC::toInt32(child.asDouble())));
-            break;
+        if (child) {
+            if (child.isNumber()) {
+                if (child.isInt32())
+                    setConstant(node, child);
+                else
+                    setConstant(node, JSValue(JSC::toInt32(child.asDouble())));
+                break;
+            }
+            if (child.isBoolean()) {
+                setConstant(node, JSValue(child.asBoolean()));
+                break;
+            }
         }