Closed Bug 1024944 Opened 10 years ago Closed 10 years ago

Replace v8 benchmarks in source tree with octane

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla33

People

(Reporter: jonco, Assigned: jonco)

Details

Attachments

(1 file)

1.82 MB, application/octet-stream
terrence
: review+
Details
There's a copy of the v8 benchmark suite in the source tree which is a little outdated now.  We should replace it with the current version of octane.
Attached file octane-bench.gzip
The patch is 11MB uncompressed, so I've gzipped it.  Some of the new benchmarks, particularly mandreel, are quite large.  Attached for reference rather than review obviously.
Attachment #8439806 - Flags: review?(terrence)
How about just deleting this directory, and people interested in benchmarking can just clone arewefastyet repo [0], to make sure to be in sync with actual tested benchmarks?

[0] https://github.com/haytjes/arewefastyet
What's nice about js/src/v8 is that we added separate run-*.js files, like run-raytrace.js. It'd be good if we could keep those...
(In reply to Jan de Mooij [:jandem] from comment #3)
Yeah, I did that :)
(In reply to Jon Coppeard (:jonco) from comment #4)
> (In reply to Jan de Mooij [:jandem] from comment #3)
> Yeah, I did that :)

Great :)
Comment on attachment 8439806 [details]
octane-bench.gzip

Makes sense.
Attachment #8439806 - Flags: review?(terrence) → review+
https://hg.mozilla.org/mozilla-central/rev/0b279a00e8b0
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: