Title: [180651] trunk/Source/_javascript_Core
Revision
180651
Author
rn...@webkit.org
Date
2015-02-25 17:24:33 -0800 (Wed, 25 Feb 2015)

Log Message

REGRESSION(r180595): construct varargs fails in FTL
https://bugs.webkit.org/show_bug.cgi?id=142030

Reviewed by Geoffrey Garen.

The bug was caused by IC size being too small for construct_varargs even though we've added a new argument.
Fixed the bug by increasing the IC size to match call_varargs.

* ftl/FTLInlineCacheSize.cpp:
(JSC::FTL::sizeOfConstructVarargs):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (180650 => 180651)


--- trunk/Source/_javascript_Core/ChangeLog	2015-02-26 01:16:21 UTC (rev 180650)
+++ trunk/Source/_javascript_Core/ChangeLog	2015-02-26 01:24:33 UTC (rev 180651)
@@ -1,3 +1,16 @@
+2015-02-25  Ryosuke Niwa  <rn...@webkit.org>
+
+        REGRESSION(r180595): construct varargs fails in FTL
+        https://bugs.webkit.org/show_bug.cgi?id=142030
+
+        Reviewed by Geoffrey Garen.
+
+        The bug was caused by IC size being too small for construct_varargs even though we've added a new argument.
+        Fixed the bug by increasing the IC size to match call_varargs.
+
+        * ftl/FTLInlineCacheSize.cpp:
+        (JSC::FTL::sizeOfConstructVarargs):
+
 2015-02-25  Mark Lam  <mark....@apple.com>
 
         ASan does not like JSC::MachineThreads::tryCopyOtherThreadStack.

Modified: trunk/Source/_javascript_Core/ftl/FTLInlineCacheSize.cpp (180650 => 180651)


--- trunk/Source/_javascript_Core/ftl/FTLInlineCacheSize.cpp	2015-02-26 01:16:21 UTC (rev 180650)
+++ trunk/Source/_javascript_Core/ftl/FTLInlineCacheSize.cpp	2015-02-26 01:24:33 UTC (rev 180651)
@@ -94,9 +94,9 @@
 size_t sizeOfConstructVarargs()
 {
 #if CPU(ARM64)
-    return 284;
+    return 300;
 #else
-    return 253;
+    return 275;
 #endif
 }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to