Skip to content

Remove single remaining threading test - #1709

Merged
alexcrichton merged 1 commit into
bytecodealliance:mainfrom
alexcrichton:remove-threading-test
Sep 10, 2026
Merged

Remove single remaining threading test#1709
alexcrichton merged 1 commit into
bytecodealliance:mainfrom
alexcrichton:remove-threading-test

Conversation

@alexcrichton

Copy link
Copy Markdown
Member

This commit removes the one component-model-threading-related test in this repository. This was added initially when threading support was first being developed as a bit of a smoke test, but nowadays wasi-libc serves as a suitable enough ground for smoke testing. This test is otherwise broken with Wasmtime's fixed behavior on *_promote intrinsics and fixing the test itself has proven a bit difficult for myself locally. The test itself also technically is a bit brittle since it's only guaranteed to work with the wasip3 target and "probably on wasip2 with optimizations", so for now just remove the brittle test.

This commit removes the one `component-model-threading`-related test in
this repository. This was added initially when threading support was
first being developed as a bit of a smoke test, but nowadays wasi-libc
serves as a suitable enough ground for smoke testing. This test is
otherwise broken with Wasmtime's fixed behavior on `*_promote`
intrinsics and fixing the test itself has proven a bit difficult for
myself locally. The test itself also technically is a bit brittle since
it's only guaranteed to work with the wasip3 target and "probably on
wasip2 with optimizations", so for now just remove the brittle test.
@alexcrichton
alexcrichton requested a review from dicej September 10, 2026 17:24
@alexcrichton
alexcrichton added this pull request to the merge queue Sep 10, 2026
Merged via the queue into bytecodealliance:main with commit 8dba3d6 Sep 10, 2026
29 checks passed
@alexcrichton
alexcrichton deleted the remove-threading-test branch September 10, 2026 18:07
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