Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

benchmark: pass execArgv to the benchmarking process #928

Closed
wants to merge 1 commit into from

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