Stream: git-wasmtime

Topic: wasmtime / PR #12907 aarch64: Fix miscompile lowering the...


view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2026 at 13:19):

alexcrichton opened PR #12907 from alexcrichton:fix-aarch64-extr to bytecodealliance:main:

This commit fixes a miscompile in the lowering of the extr instruction for the aarch64 backend where one of the shift operands is 0. In this edge case the generated extr instruction did not match the input CLIF semantics, calculating a different value. The fix here is to only use the extr instruction when both immediates are larger than 0.

<!--
Please make sure you include the following information:

Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.html

Please ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->

view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2026 at 13:19):

alexcrichton requested wasmtime-compiler-reviewers for a review on PR #12907.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2026 at 13:19):

alexcrichton requested fitzgen for a review on PR #12907.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2026 at 13:19):

alexcrichton requested wasmtime-core-reviewers for a review on PR #12907.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2026 at 15:42):

cfallin submitted PR review:

Good find!

view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2026 at 15:42):

cfallin added PR #12907 aarch64: Fix miscompile lowering the extr instruction to the merge queue.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2026 at 16:24):

cfallin merged PR #12907.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2026 at 16:24):

cfallin removed PR #12907 aarch64: Fix miscompile lowering the extr instruction from the merge queue.


Last updated: Apr 12 2026 at 23:10 UTC