diff options
author | Andreas Madsen <amwebdk@gmail.com> | 2016-10-12 20:28:57 +0200 |
---|---|---|
committer | James M Snell <jasnell@gmail.com> | 2016-10-17 08:57:28 -0700 |
commit | 41173de77b04ec9b659dc59d6b448556147a410e (patch) | |
tree | 2128525d6377b403193cf4809e85d40f1f024862 | |
parent | 138a7922ce8bc92aa1c6e841a2e713b6371ffbc5 (diff) | |
download | android-node-v8-41173de77b04ec9b659dc59d6b448556147a410e.tar.gz android-node-v8-41173de77b04ec9b659dc59d6b448556147a410e.tar.bz2 android-node-v8-41173de77b04ec9b659dc59d6b448556147a410e.zip |
benchmark: change the execution order
This changes the execution order from "iter, file, binary" to "file,
iter, binary". This means the csv no longer has to buffered completely.
This also has the added effect that stopping compare.js early or
interfering with performance only affects a single benchmark, instead of
all of them.
Refs: https://github.com/nodejs/node/issues/8659
PR-URL: https://github.com/nodejs/node/pull/9064
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
-rw-r--r-- | benchmark/compare.js | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/benchmark/compare.js b/benchmark/compare.js index de328d60fc..ea431b18cb 100644 --- a/benchmark/compare.js +++ b/benchmark/compare.js @@ -40,8 +40,8 @@ if (benchmarks.length === 0) { // Create queue from the benchmarks list such both node versions are tested // `runs` amount of times each. const queue = []; -for (let iter = 0; iter < runs; iter++) { - for (const filename of benchmarks) { +for (const filename of benchmarks) { + for (let iter = 0; iter < runs; iter++) { for (const binary of binaries) { queue.push({ binary, filename, iter }); } |