-
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
Allow wasm object files as input, even when WASM_OBJECT_FILES=0 is set #8600
Conversation
d05e3ce
to
d92e429
Compare
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.
Is the opposite also true - can we mix wasm and bitcode files with WASM_OBJECT_FILES=1?
Please add some mention of this on the WASM_OBJECT_FILES section in src/settings.js (not sure where's a better place).
Please add a test here.
Yes. WASM_OBJECT_FILES at compile time means "please output a wasm object file". WASM_OBJECT_FILES at link time means "use libraries in the form of wasm object files". In the long run both of these should be replaced by some kind of LTO flag. |
While fixing this I found a bug in our LTO implementation. Fixed upstream: https://reviews.llvm.org/D61924. Will need to wait for the waterfall to get a clean build of that before this change can lang I think. |
Specifying WASM_OBJECT_FILES=0 on the link line is current needed if one want to select LTO libraries. However this should not exclude some input being wasm object files. Fixes #8599
d92e429
to
993c990
Compare
emscripten-core#8600) Specifying WASM_OBJECT_FILES=0 on the link line is current needed if one want to select LTO libraries. However this should not exclude some input being wasm object files. Fixes emscripten-core#8599
…=0 is set (emscripten-core#8600)" This reverts commit 4aa04c5.
…=0 is set (emscripten-core#8600)" This reverts commit 4aa04c5.
emscripten-core#8600) Specifying WASM_OBJECT_FILES=0 on the link line is current needed if one want to select LTO libraries. However this should not exclude some input being wasm object files. Fixes emscripten-core#8599
Specifying WASM_OBJECT_FILES=0 on the link line is current needed if
one want to select LTO libraries. However this should not exclude
some input being wasm object files.
Fixes #8599