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

[request] emsdk/4.0.2 #26509

Open
hieunguyenbv opened this issue Feb 3, 2025 · 1 comment
Open

[request] emsdk/4.0.2 #26509

hieunguyenbv opened this issue Feb 3, 2025 · 1 comment

Comments

@hieunguyenbv
Copy link

Package Name/Version

emsdk/4.0.2

Changelog

https://github.com/emscripten-core/emsdk/releases/tag/4.0.2

Context about the new update

The above-mentioned version is newly released by the upstream project and not yet available as a recipe.
Please add this version.

@hieunguyenbv
Copy link
Author

Hi all,
I noticed that Conan Center provides newer EMSDK versions (3.1.72 and 3.1.73), but both versions fail when running Binaryen optimizations (wasm-opt). The error message indicates that wasm-opt does not recognize certain options, specifically:

Unknown option '--enable-bulk-memory-opt'
Or
Unknown option '--enable-call-indirect-overlong'

Error log:

link:DEBUG: using binaryen
Unknown option '--enable-call-indirect-overlong'
em++: error: '/home/l146/.conan2/p/emsdkc9782d7aefec5/p/bin/upstream/bin/wasm-opt --strip-target-features --post-emscripten --pass-arg=post-emscripten-side-module -O3 --low-memory-unused --asyncify --pass-arg=asyncify-relocatable --pass-arg=asyncify-propagate-addlist [email protected]_,env.asyncjs,.fd_sync,.emscripten_promise_await,._dlopen_js,.emscripten_idb_load,.emscripten_idb_store,.emscripten_idb_delete,.emscripten_idb_exists,.emscripten_idb_clear,.emscripten_idb_load_blob,.emscripten_idb_store_blob,.emscripten_sleep,.emscripten_wget_data,.emscripten_scan_registers,.emscripten_lazy_load_code,._load_secondary_module,.emscripten_fiber_swap,*.SDL_Delay --pass-arg=directize-initial-contents-immutable --no-stack-ir libRFWARM.wasm -o libRFWARM.wasm --mvp-features --enable-threads --enable-bulk-memory --enable-call-indirect-overlong --enable-multivalue --enable-mutable-globals --enable-reference-types --enable-sign-ext' failed (returned 1)

This issue is resolved in EMSDK 3.1.74 or newer (tested with 3.1.74 and 4.0.2), where the options are correctly recognized.

A similar issue was reported here: iree-org/iree#19425.

Would it be possible to update the Conan Center package to use a newer EMSDK version (suggest 4.0.2)? This would prevent similar build failures for users relying on the prebuilt versions.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant