Skip to content

Migrate the Emscripten benchmarks to wasi-sdk#321

Merged
cfallin merged 1 commit into
bytecodealliance:mainfrom
fitzgen:wasi-sdk-instead-of-emscripten
Jun 25, 2026
Merged

Migrate the Emscripten benchmarks to wasi-sdk#321
cfallin merged 1 commit into
bytecodealliance:mainfrom
fitzgen:wasi-sdk-instead-of-emscripten

Conversation

@fitzgen

@fitzgen fitzgen commented Jun 25, 2026

Copy link
Copy Markdown
Member

noop, blake3-simd, and intgemm-simd were the only benchmarks still built with Emscripten. They now build with wasi-sdk, matching the rest of the C/C++ benchmarks and removing the Emscripten dependency.

noop was straightforward since it doesn't do anything.

blake3-simd and intgemm-simd were trickier. Emscripten comes with shims for mapping x86 SSE intrinsics onto Wasm SIMD, but wasi-sdk doesn't do that, so this commit introduces wasm_sse_compat.h headers to fill that gap ourselves. Note that blake3-simd and intgemm-simd use almost non-overlapping sets of SIMD instructions, and one is C and the other C++, so I didn't bother trying to share a single wasm_sse_compat.h file between them.

`noop`, `blake3-simd`, and `intgemm-simd` were the only benchmarks still built
with Emscripten. They now build with `wasi-sdk`, matching the rest of the C/C++
benchmarks and removing the Emscripten dependency.

`noop` was straightforward since it doesn't do anything.

`blake3-simd` and `intgemm-simd` were trickier. Emscripten comes with shims for
mapping x86 SSE intrinsics onto Wasm SIMD, but `wasi-sdk` doesn't do that, so
this commit introduces `wasm_sse_compat.h` headers to fill that gap
ourselves. Note that `blake3-simd` and `intgemm-simd` use almost non-overlapping
sets of SIMD instructions, and one is C and the other C++, so I didn't bother
trying to share a single `wasm_sse_compat.h` file between them.
@fitzgen fitzgen requested a review from cfallin June 25, 2026 18:46

@cfallin cfallin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@cfallin cfallin merged commit 98ff8a5 into bytecodealliance:main Jun 25, 2026
12 checks passed
@fitzgen fitzgen deleted the wasi-sdk-instead-of-emscripten branch June 25, 2026 21:50
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.

2 participants