-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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 BINARYEN_FEATURES, and use that to pass clang feature flags to binaryen #8403
Conversation
The test here may not pass depending on what node version we have on the bots - if it fails I'll remove it. |
This now just removes the feature section code, so that we can just rely on the features section. |
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.
Yay!
Rewritten to be feature section friendly, and now it expects to get the list of features as |
Also updates binaryen to version_77, and adds a TODO test (which passes on v8 master, but not node lts yet). |
Gets the list of features as metadata['features'] from wasm-emscripten-finalize. It then passes those to the various invocations we do of binaryen tools. Also updates binaryen to version_77 for the binaryen side of things, and adds a TODO test (which passes on v8 master, but not node LTS yet).
This reverts commit 9e2efb0.
This reverts commit 9e2efb0.
Gets the list of features as metadata['features'] from wasm-emscripten-finalize. It then passes those to the various invocations we do of binaryen tools. Also updates binaryen to version_77 for the binaryen side of things, and adds a TODO test (which passes on v8 master, but not node LTS yet).
This allows using nontrapping float to int, which makes e.g. fasta 10% faster or so.