Skip to content

Adjust some documentation for threading intrinsics#674

Merged
lukewagner merged 2 commits into
WebAssembly:mainfrom
alexcrichton:adjust-threading-things
Jul 1, 2026
Merged

Adjust some documentation for threading intrinsics#674
lukewagner merged 2 commits into
WebAssembly:mainfrom
alexcrichton:adjust-threading-things

Conversation

@alexcrichton

Copy link
Copy Markdown
Collaborator

In WebAssembly#656 the threading-related intrinsics were revised and updated, but
`Binary.md` wasn't updated in the same PR. This commit removes the
removed intrinsics (`thread.switch-to` and `thread.yield-to`),
reorganizes the list slightly (moves `thread.yield` down), and then
renumbers the remaining intrinsics.
This matches what wasm-tools/Wasmtime do at least which is
`thread.yield` is supported with just the component-model-async feature
and doesn't need component-model-threading. IIRC this is due to the fact
of this intrinsic existing prior to component-model-threading and it was
just renamed later on.
alexcrichton added a commit to alexcrichton/wasm-tools that referenced this pull request Jul 1, 2026
This commit synchronizes `cm-threading` intrinsics parsed by
wasmparser/tooling/etc to match the current specification. These
intentionally diverged from the specification in bytecodealliance#2430 and when the
specification was updated in WebAssembly/component-model#656 it ended up
settling on some minor adjustments. This additionally takes into account
WebAssembly/component-model#674 for binary parsing adjustments.
@lukewagner

Copy link
Copy Markdown
Member

Great catches, thanks!

@lukewagner lukewagner merged commit 415020e into WebAssembly:main Jul 1, 2026
1 check passed
github-merge-queue Bot pushed a commit to bytecodealliance/wasm-tools that referenced this pull request Jul 1, 2026
This commit synchronizes `cm-threading` intrinsics parsed by
wasmparser/tooling/etc to match the current specification. These
intentionally diverged from the specification in #2430 and when the
specification was updated in WebAssembly/component-model#656 it ended up
settling on some minor adjustments. This additionally takes into account
WebAssembly/component-model#674 for binary parsing adjustments.
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