Switch the value profiler back to 8 buckets, because we suspect that while this
authorfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 7 Nov 2011 23:08:54 +0000 (23:08 +0000)
committerfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 7 Nov 2011 23:08:54 +0000 (23:08 +0000)
is more expensive it's also more stable.

Rubber stamped by Geoff Garen.

* bytecode/ValueProfile.h:

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

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

index 27bd1d7..76fc072 100644 (file)
@@ -1,3 +1,12 @@
+2011-11-07  Filip Pizlo  <fpizlo@apple.com>
+
+        Switch the value profiler back to 8 buckets, because we suspect that while this
+        is more expensive it's also more stable.
+
+        Rubber stamped by Geoff Garen.
+
+        * bytecode/ValueProfile.h:
+
 2011-11-07  Andrew Wason  <rectalogic@rectalogic.com>
 
         Uninitialized Heap member var
index b7e920e..ec4ea71 100644 (file)
@@ -38,7 +38,7 @@ namespace JSC {
 
 #if ENABLE(VALUE_PROFILER)
 struct ValueProfile {
-    static const unsigned logNumberOfBuckets = 0; // 1 bucket
+    static const unsigned logNumberOfBuckets = 3; // 8 buckets
     static const unsigned numberOfBuckets = 1 << logNumberOfBuckets;
     static const unsigned numberOfSpecFailBuckets = 1;
     static const unsigned bucketIndexMask = numberOfBuckets - 1;