-
Notifications
You must be signed in to change notification settings - Fork 465
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
Add benchmarks #623
Add benchmarks #623
Conversation
I'm also pretty sure it factors out the settings needed for turning on/off exceptions in an addon's binding.gyp. The except.gypi and noexcept.gypi files might be useful in and of themselves. If they are, we should add lines to index.js which gives their path for inclusion in addons. |
f03a98c
to
e92ef1a
Compare
e92ef1a
to
caf97a5
Compare
Alas, gyp does not support variable resolution for ...
'includes': [ '<(variable_will_not_work_here)']
... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM just one comment.
@NickNaso I linked benchmark/README.md from the main README.md. |
Built products in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Adds the framework for writing benchmarks and two basic benchmarks. PR-URL: #623 Reviewed-By: Nicola Del Gobbo <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
Landed in 6a06463. |
Adds the framework for writing benchmarks and two basic benchmarks. PR-URL: nodejs/node-addon-api#623 Reviewed-By: Nicola Del Gobbo <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
Adds the framework for writing benchmarks and two basic benchmarks. PR-URL: nodejs/node-addon-api#623 Reviewed-By: Nicola Del Gobbo <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
Adds the framework for writing benchmarks and two basic benchmarks. PR-URL: nodejs/node-addon-api#623 Reviewed-By: Nicola Del Gobbo <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
Adds the framework for writing benchmarks and two basic benchmarks. PR-URL: nodejs/node-addon-api#623 Reviewed-By: Nicola Del Gobbo <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
This adds a benchmarking framework and two benchmarks: one for function calls and one for property descriptor getter/setter calls.