Skip to content

Disallow fewer instructions in Wizer#11851

Merged
fitzgen merged 1 commit intobytecodealliance:mainfrom
alexcrichton:wizer-more-instructions
Oct 14, 2025
Merged

Disallow fewer instructions in Wizer#11851
fitzgen merged 1 commit intobytecodealliance:mainfrom
alexcrichton:wizer-more-instructions

Conversation

@alexcrichton
Copy link
Member

Specifically allow table.get, ref.null, ref.is_null, select (typed), ref.func, and table.size. These are fine to execute and the incompatibility with Wizer instead comes when a table or a reference type global is mutated. That mutation is disallowed elsewhere, however, so there's no need to reject these instructions.

Specifically allow `table.get`, `ref.null`, `ref.is_null`, `select`
(typed), `ref.func`, and `table.size`. These are fine to execute and the
incompatibility with Wizer instead comes when a table or a reference
type global is mutated. That mutation is disallowed elsewhere, however,
so there's no need to reject these instructions.
@alexcrichton alexcrichton requested a review from a team as a code owner October 14, 2025 16:40
@alexcrichton alexcrichton requested review from fitzgen and removed request for a team October 14, 2025 16:40
@fitzgen fitzgen added this pull request to the merge queue Oct 14, 2025
Merged via the queue into bytecodealliance:main with commit ab2486b Oct 14, 2025
44 checks passed
@alexcrichton alexcrichton deleted the wizer-more-instructions branch October 14, 2025 17:48
bongjunj pushed a commit to prosyslab/wasmtime that referenced this pull request Oct 20, 2025
Specifically allow `table.get`, `ref.null`, `ref.is_null`, `select`
(typed), `ref.func`, and `table.size`. These are fine to execute and the
incompatibility with Wizer instead comes when a table or a reference
type global is mutated. That mutation is disallowed elsewhere, however,
so there's no need to reject these instructions.
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