Stream: git-wasmtime

Topic: wasmtime / PR #14631 Cranelift: ISLE: opt: add missing `t...


view this post on Zulip Wasmtime GitHub notifications bot (Oct 09 2026 at 18:10):

avanhatt opened PR #14631 from avanhatt:i32x2-scalar-guards to bytecodealliance:main:

With changes to include more vector logic, the ISLE verifier found several more cases related to #14566 where mid-end rules are not correctly guarded to exclude vector 64-bit vector types such as i32x2. Theiconst_s and iconst_u terms handle scalars, i128, and 128-bit vectors, so attempts to build 64-bit vector constants fail to match and cause compiler panics.

This PR adds filetests for each and fixes with ty_int guards. Fixes #14566. Will put up verifier/spec changes as separate PR.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 09 2026 at 18:10):

avanhatt requested wasmtime-compiler-reviewers for a review on PR #14631.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 09 2026 at 18:10):

avanhatt requested fitzgen for a review on PR #14631.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 09 2026 at 19:46):

github-actions[bot] added the label cranelift on PR #14631.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 09 2026 at 19:46):

github-actions[bot] added the label isle on PR #14631.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 09 2026 at 19:57):

:thumbs_up: cfallin submitted PR review:

Thanks!

view this post on Zulip Wasmtime GitHub notifications bot (Oct 09 2026 at 19:57):

cfallin added PR #14631 Cranelift: ISLE: opt: add missing ty_int guards to 6 rules to the merge queue.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 09 2026 at 20:50):

github-actions[bot] commented on PR #14631:

Subscribe to Label Action

cc @avanhatt, @cfallin, @fitzgen, @mmcloughlin

<details>
This issue or pull request has been labeled: "cranelift", "isle"

Thus the following users have been cc'd because of the following labels:

To subscribe or unsubscribe from this label, edit the <code>.github/subscribe-to-label.json</code> configuration file.

Learn more.
</details>

view this post on Zulip Wasmtime GitHub notifications bot (Oct 09 2026 at 21:56):

cfallin removed PR #14631 Cranelift: ISLE: opt: add missing ty_int guards to 6 rules from the merge queue.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 09 2026 at 21:56):

:check: cfallin merged PR #14631.


Last updated: Oct 11 2026 at 04:10 UTC