Author: [email protected]
Date: Tue May 26 06:28:51 2009
New Revision: 2058
Modified:
branches/bleeding_edge/benchmarks/README.txt
branches/bleeding_edge/benchmarks/base.js
branches/bleeding_edge/benchmarks/revisions.html
Log:
Remove duplicate line in benchmarks/base.js
This resolves issue 356 (http://code.google.com/p/v8/issues/detail?id=356)
thanks to bitRAKE. Changed benchmark version number from 4 to 5 resulting
in updates to base.js, README.txt and revisions.html.
Review URL: http://codereview.chromium.org/113839
Modified: branches/bleeding_edge/benchmarks/README.txt
==============================================================================
--- branches/bleeding_edge/benchmarks/README.txt (original)
+++ branches/bleeding_edge/benchmarks/README.txt Tue May 26 06:28:51 2009
@@ -52,3 +52,9 @@
Furthermore, all the unused parts of the Prototype library were
removed from the RayTrace benchmark. This does not affect the running
of the benchmark.
+
+
+Changes from Version 4 to Version 5
+===================================
+
+Removed duplicate line in random seed code.
Modified: branches/bleeding_edge/benchmarks/base.js
==============================================================================
--- branches/bleeding_edge/benchmarks/base.js (original)
+++ branches/bleeding_edge/benchmarks/base.js Tue May 26 06:28:51 2009
@@ -78,7 +78,7 @@
// Scores are not comparable across versions. Bump the version if
// you're making changes that will affect that scores, e.g. if you add
// a new benchmark or change an existing one.
-BenchmarkSuite.version = '4';
+BenchmarkSuite.version = '5';
// To make the benchmark results predictable, we replace Math.random
@@ -90,7 +90,6 @@
seed = ((seed + 0x7ed55d16) + (seed << 12)) & 0xffffffff;
seed = ((seed ^ 0xc761c23c) ^ (seed >>> 19)) & 0xffffffff;
seed = ((seed + 0x165667b1) + (seed << 5)) & 0xffffffff;
- seed = ((seed + 0xd3a2646c) ^ (seed << 9)) & 0xffffffff;
seed = ((seed + 0xd3a2646c) ^ (seed << 9)) & 0xffffffff;
seed = ((seed + 0xfd7046c5) + (seed << 3)) & 0xffffffff;
seed = ((seed ^ 0xb55a4f09) ^ (seed >>> 16)) & 0xffffffff;
Modified: branches/bleeding_edge/benchmarks/revisions.html
==============================================================================
--- branches/bleeding_edge/benchmarks/revisions.html (original)
+++ branches/bleeding_edge/benchmarks/revisions.html Tue May 26 06:28:51
2009
@@ -20,6 +20,11 @@
</p>
+<div class="subtitle"><h3>Version 5 (<a
href="http://v8.googlecode.com/svn/data/benchmarks/v5/run.html";>link</a>)</h3></div>
+
+<p>Removed a duplicate line in the base random seed code.
+</p>
+
<div class="subtitle"><h3>Version 4 (<a
href="http://v8.googlecode.com/svn/data/benchmarks/v4/run.html";>link</a>)</h3></div>
<p>The <i>Splay</i> benchmark is a newcomer in version 4. It
--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---
