Switch back to 1+1 value profiling buckets, since it didn't help on arewefastyet,
authorfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 8 Nov 2011 03:02:19 +0000 (03:02 +0000)
committerfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 8 Nov 2011 03:02:19 +0000 (03:02 +0000)
but it appears to help on other benchmarks.

Rubber stamped by Oliver Hunt.

* bytecode/ValueProfile.h:

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

Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/bytecode/ValueProfile.h

index b985caa..70713e7 100644 (file)
@@ -1,3 +1,12 @@
+2011-11-07  Filip Pizlo  <fpizlo@apple.com>
+
+        Switch back to 1+1 value profiling buckets, since it didn't help on arewefastyet,
+        but it appears to help on other benchmarks.
+
+        Rubber stamped by Oliver Hunt.
+
+        * bytecode/ValueProfile.h:
+
 2011-11-07  Ariya Hidayat  <ariya@sencha.com>
 
         "use strict" can not contain escape sequences or line continuation
index ec4ea71..b7e920e 100644 (file)
@@ -38,7 +38,7 @@ namespace JSC {
 
 #if ENABLE(VALUE_PROFILER)
 struct ValueProfile {
-    static const unsigned logNumberOfBuckets = 3; // 8 buckets
+    static const unsigned logNumberOfBuckets = 0; // 1 bucket
     static const unsigned numberOfBuckets = 1 << logNumberOfBuckets;
     static const unsigned numberOfSpecFailBuckets = 1;
     static const unsigned bucketIndexMask = numberOfBuckets - 1;