Check to see how the perf bots react to megamorphic load being disabled.
authorfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 18 Apr 2016 20:28:51 +0000 (20:28 +0000)
committerfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 18 Apr 2016 20:28:51 +0000 (20:28 +0000)
Rubber stamped by Chris Dumez.

* runtime/Options.h:

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

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

index 2319029..5e5296f 100644 (file)
@@ -1,3 +1,11 @@
+2016-04-18  Filip Pizlo  <fpizlo@apple.com>
+
+        Check to see how the perf bots react to megamorphic load being disabled.
+
+        Rubber stamped by Chris Dumez.
+
+        * runtime/Options.h:
+
 2016-04-18  Keith Miller  <keith_miller@apple.com>
 
         We should support delete in the DFG
index 2ba5aea..dd91922 100644 (file)
@@ -192,7 +192,7 @@ typedef const char* optionString;
     v(bool, clobberAllRegsInFTLICSlowPath, !ASSERT_DISABLED, nullptr) \
     v(bool, useAccessInlining, true, nullptr) \
     v(unsigned, maxAccessVariantListSize, 13, nullptr) \
-    v(unsigned, megamorphicLoadCost, 10, nullptr) \
+    v(unsigned, megamorphicLoadCost, 999, nullptr) /* This used to be 10, but we're temporarily testing what happens when the feature is disabled. */\
     v(bool, usePolyvariantDevirtualization, true, nullptr) \
     v(bool, usePolymorphicAccessInlining, true, nullptr) \
     v(bool, usePolymorphicCallInlining, true, nullptr) \