Author: [EMAIL PROTECTED]
Date: Tue Dec  2 05:02:11 2008
New Revision: 895

Modified:
    branches/bleeding_edge/benchmarks/base.js
    branches/bleeding_edge/benchmarks/run.html
    branches/bleeding_edge/benchmarks/run.js

Log:
Changed benchmark runners to always display at least three significant
digits in the results.


Modified: branches/bleeding_edge/benchmarks/base.js
==============================================================================
--- branches/bleeding_edge/benchmarks/base.js   (original)
+++ branches/bleeding_edge/benchmarks/base.js   Tue Dec  2 05:02:11 2008
@@ -120,7 +120,7 @@
      }
      if (runner.NotifyScore) {
        var score = BenchmarkSuite.GeometricMean(BenchmarkSuite.scores);
-      runner.NotifyScore(Math.round(100 * score));
+      runner.NotifyScore(100 * score);
      }
    }
    RunStep();
@@ -164,7 +164,7 @@
    var score = this.reference / mean;
    BenchmarkSuite.scores.push(score);
    if (this.runner.NotifyResult) {
-    this.runner.NotifyResult(this.name, Math.round(100 * score));
+    this.runner.NotifyResult(this.name, 100 * score);
    }
  }

@@ -218,4 +218,15 @@
      return null;
    }
    return RunNext();
+}
+
+
+// Converts a score value to a string with at least three significant
+// digits.
+function formatScore(value) {
+  if (value > 100) {
+    return value.toFixed(0);
+  } else {
+    return value.toPrecision(3);
+  }
  }

Modified: branches/bleeding_edge/benchmarks/run.html
==============================================================================
--- branches/bleeding_edge/benchmarks/run.html  (original)
+++ branches/bleeding_edge/benchmarks/run.html  Tue Dec  2 05:02:11 2008
@@ -21,7 +21,7 @@


  function AddResult(name, result) {
-  var text = name + ': ' + result;
+  var text = name + ': ' + formatScore(result);
    var results = document.getElementById("results");
    results.innerHTML += (text + "<br/>");
  }
@@ -36,7 +36,7 @@
  function AddScore(score) {
    var status = document.getElementById("status");
    if (success) {
-    status.innerHTML = "Score: " + score;
+    status.innerHTML = "Score: " + formatScore(score);
    }
  }


Modified: branches/bleeding_edge/benchmarks/run.js
==============================================================================
--- branches/bleeding_edge/benchmarks/run.js    (original)
+++ branches/bleeding_edge/benchmarks/run.js    Tue Dec  2 05:02:11 2008
@@ -36,7 +36,7 @@
  var success = true;

  function PrintResult(name, result) {
-  print(name + ': ' + result);
+  print(name + ': ' + formatScore(result));
  }


@@ -49,7 +49,8 @@
  function PrintScore(score) {
    if (success) {
      print('----');
-    print('Score (version ' + BenchmarkSuite.version + '): ' + score);
+    print('Score (version ' + BenchmarkSuite.version + '): '
+          + formatScore(score));
    }
  }


--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to