Reduce thresholds that control the maximum IC stub size.
authorfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 5 May 2016 21:53:27 +0000 (21:53 +0000)
committerfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 5 May 2016 21:53:27 +0000 (21:53 +0000)
Rubber stamped by Chris Dumez and Benjamin Poulain.

This reduces the thresholds to before the megamorphic load optimizations to see if that
recovers a PLT regression.

* runtime/Options.h:

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

Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/runtime/Options.h

index b4469b0..6130b31 100644 (file)
@@ -1,5 +1,16 @@
 2016-05-05  Filip Pizlo  <fpizlo@apple.com>
 
+        Reduce thresholds that control the maximum IC stub size.
+
+        Rubber stamped by Chris Dumez and Benjamin Poulain.
+        
+        This reduces the thresholds to before the megamorphic load optimizations to see if that
+        recovers a PLT regression.
+
+        * runtime/Options.h:
+
+2016-05-05  Filip Pizlo  <fpizlo@apple.com>
+
         We shouldn't crash if DFG AI proved that something was unreachable on one run but then decided not to prove it on another run
         https://bugs.webkit.org/show_bug.cgi?id=157379
 
index a31eeda..6ae3ce4 100644 (file)
@@ -123,9 +123,9 @@ typedef const char* optionString;
     v(bool, forceCodeBlockLiveness, false, Normal, nullptr) \
     v(bool, forceICFailure, false, Normal, nullptr) \
     \
-    v(unsigned, repatchCountForCoolDown, 10, Normal, nullptr) \
+    v(unsigned, repatchCountForCoolDown, 8, Normal, nullptr) \
     v(unsigned, initialCoolDownCount, 20, Normal, nullptr) \
-    v(unsigned, repatchBufferingCountdown, 10, Normal, nullptr) \
+    v(unsigned, repatchBufferingCountdown, 8, Normal, nullptr) \
     \
     v(bool, dumpGeneratedBytecodes, false, Normal, nullptr) \
     v(bool, dumpBytecodeLivenessResults, false, Normal, nullptr) \
@@ -191,7 +191,7 @@ typedef const char* optionString;
     v(bool, ftlCrashes, false, Normal, nullptr) /* fool-proof way of checking that you ended up in the FTL. ;-) */\
     v(bool, clobberAllRegsInFTLICSlowPath, !ASSERT_DISABLED, Normal, nullptr) \
     v(bool, useAccessInlining, true, Normal, nullptr) \
-    v(unsigned, maxAccessVariantListSize, 13, Normal, nullptr) \
+    v(unsigned, maxAccessVariantListSize, 8, Normal, nullptr) \
     v(unsigned, megamorphicLoadCost, 999, Normal, nullptr) /* This used to be 10, but we're temporarily testing what happens when the feature is disabled. */\
     v(bool, usePolyvariantDevirtualization, true, Normal, nullptr) \
     v(bool, usePolymorphicAccessInlining, true, Normal, nullptr) \