Title: [191753] trunk/Source/_javascript_Core
Revision
191753
Author
[email protected]
Date
2015-10-29 14:08:05 -0700 (Thu, 29 Oct 2015)

Log Message

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):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (191752 => 191753)


--- trunk/Source/_javascript_Core/ChangeLog	2015-10-29 20:51:28 UTC (rev 191752)
+++ trunk/Source/_javascript_Core/ChangeLog	2015-10-29 21:08:05 UTC (rev 191753)
@@ -1,3 +1,16 @@
+2015-10-29  Mark Lam  <[email protected]>
+
+        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  <[email protected]>
 
         Unreviewed, fix iOS build.

Modified: trunk/Source/_javascript_Core/ftl/FTLCapabilities.cpp (191752 => 191753)


--- trunk/Source/_javascript_Core/ftl/FTLCapabilities.cpp	2015-10-29 20:51:28 UTC (rev 191752)
+++ trunk/Source/_javascript_Core/ftl/FTLCapabilities.cpp	2015-10-29 21:08:05 UTC (rev 191753)
@@ -84,7 +84,6 @@
     case StrCat:
     case ArithAdd:
     case ArithClz32:
-    case ArithSub:
     case ArithMul:
     case ArithDiv:
     case ArithMod:
@@ -212,6 +211,10 @@
     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
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to