Title: [264942] trunk/Source/WTF
Revision
264942
Author
ysuz...@apple.com
Date
2020-07-27 13:30:03 -0700 (Mon, 27 Jul 2020)

Log Message

[JSC] Disable BIGINT32 optimization temporarily because of SP2 / JS2 regression
https://bugs.webkit.org/show_bug.cgi?id=214776

Reviewed by Darin Adler.

We disable BIGINT32 optimization because it causes 1% regression in JetStream2 and Speedometer2.
We should enable it after we fix the cause of regression.

* wtf/PlatformUse.h:

Modified Paths

Diff

Modified: trunk/Source/WTF/ChangeLog (264941 => 264942)


--- trunk/Source/WTF/ChangeLog	2020-07-27 20:25:05 UTC (rev 264941)
+++ trunk/Source/WTF/ChangeLog	2020-07-27 20:30:03 UTC (rev 264942)
@@ -1,3 +1,15 @@
+2020-07-27  Yusuke Suzuki  <ysuz...@apple.com>
+
+        [JSC] Disable BIGINT32 optimization temporarily because of SP2 / JS2 regression
+        https://bugs.webkit.org/show_bug.cgi?id=214776
+
+        Reviewed by Darin Adler.
+
+        We disable BIGINT32 optimization because it causes 1% regression in JetStream2 and Speedometer2.
+        We should enable it after we fix the cause of regression.
+
+        * wtf/PlatformUse.h:
+
 2020-07-27  Carlos Garcia Campos  <cgar...@igalia.com>
 
         [WPE][GTK] Add API to support "Privacy Report"

Modified: trunk/Source/WTF/wtf/PlatformUse.h (264941 => 264942)


--- trunk/Source/WTF/wtf/PlatformUse.h	2020-07-27 20:25:05 UTC (rev 264941)
+++ trunk/Source/WTF/wtf/PlatformUse.h	2020-07-27 20:30:03 UTC (rev 264942)
@@ -142,7 +142,9 @@
 #endif
 
 #if USE(JSVALUE64)
-#define USE_BIGINT32 1
+/* FIXME: Enable BIGINT32 optimization again after we ensure Speedometer2 and JetStream2 regressions are fixed. */
+/* https://bugs.webkit.org/show_bug.cgi?id=214777 */
+#define USE_BIGINT32 0
 #endif
 
 /* FIXME: This name should be more specific if it is only for use with CallFrame* */
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to