cdjs-tests.yaml/main.js.ftl fails due to FTL ArithSub code for supporting UntypedUse...
authormark.lam@apple.com <mark.lam@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 29 Oct 2015 21:08:05 +0000 (21:08 +0000)
committermark.lam@apple.com <mark.lam@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 29 Oct 2015 21:08:05 +0000 (21:08 +0000)
https://bugs.webkit.org/show_bug.cgi?id=150687

Unreviewed.

Disabling the feature while it is being debugged.  I'm doing this by effectively
rolling out only the changes in FTLCapabilities.cpp.

* ftl/FTLCapabilities.cpp:
(JSC::FTL::canCompile):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@191753 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/ftl/FTLCapabilities.cpp

index ec468c3..d1f5a66 100644 (file)
@@ -1,3 +1,16 @@
+2015-10-29  Mark Lam  <mark.lam@apple.com>
+
+        cdjs-tests.yaml/main.js.ftl fails due to FTL ArithSub code for supporting UntypedUse operands.
+        https://bugs.webkit.org/show_bug.cgi?id=150687
+
+        Unreviewed.
+
+        Disabling the feature while it is being debugged.  I'm doing this by effectively
+        rolling out only the changes in FTLCapabilities.cpp.
+
+        * ftl/FTLCapabilities.cpp:
+        (JSC::FTL::canCompile):
+
 2015-10-29  Filip Pizlo  <fpizlo@apple.com>
 
         Unreviewed, fix iOS build.
index e327f76..5dd7632 100644 (file)
@@ -84,7 +84,6 @@ inline CapabilityLevel canCompile(Node* node)
     case StrCat:
     case ArithAdd:
     case ArithClz32:
-    case ArithSub:
     case ArithMul:
     case ArithDiv:
     case ArithMod:
@@ -212,6 +211,10 @@ inline CapabilityLevel canCompile(Node* node)
     case PutSetterByVal:
         // These are OK.
         break;
+    case ArithSub:
+        if (node->result() == NodeResultJS)
+            return CannotCompile;
+        break;
 
     case Identity:
         // No backend handles this because it will be optimized out. But we may check