Skip to content

Conversation

petkaantonov
Copy link
Contributor

Benchmarker should pass exec flags (e.g. --no-crankshaft,
--turbofan-filter, --trace-opt etc) to the benchmarking process

Benchmarker should pass exec flags (e.g. --no-crankshaft,
--turbofan-filter, --trace-opt etc) to the benchmarking process
@sam-github
Copy link
Contributor

+1, @bajtos had to do this in #173, too (see a930ddb).

I'll merge this in a day or so unless someone comes up with a reason not to.

@chrisdickinson
Copy link
Contributor

Feel free to merge now, I think this should be a relatively innocuous change.

@bajtos
Copy link
Contributor

bajtos commented Feb 24, 2015

👍

sam-github pushed a commit to sam-github/node that referenced this pull request Feb 25, 2015
Benchmarker should pass exec flags (e.g. --no-crankshaft,
--turbofan-filter, --trace-opt etc) to the benchmarking process

Reviewed-By: Sam Roberts <[email protected]>
Reviewed-By: Chris Dickinson <[email protected]>
PR-URL: nodejs#928
@sam-github
Copy link
Contributor

Landed in 8a1e22a, thank you.

@sam-github sam-github closed this Feb 25, 2015
petkaantonov added a commit to petkaantonov/io.js that referenced this pull request Feb 25, 2015
Benchmarker should pass exec flags (e.g. --no-crankshaft,
--turbofan-filter, --trace-opt etc) to the benchmarking process

Reviewed-By: Sam Roberts <[email protected]>
Reviewed-By: Chris Dickinson <[email protected]>
PR-URL: nodejs#928
@rvagg rvagg mentioned this pull request Feb 25, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants