Skip to content

winch(aarch64): v128 ABI plumbing#13948

Merged
alexcrichton merged 1 commit into
bytecodealliance:mainfrom
macovedj:winch-aarch64-v128-abi
Jul 24, 2026
Merged

winch(aarch64): v128 ABI plumbing#13948
alexcrichton merged 1 commit into
bytecodealliance:mainfrom
macovedj:winch-aarch64-v128-abi

Conversation

@macovedj

Copy link
Copy Markdown
Contributor

While observing which tests flip as we add simd instructions for #9925, it became clear that while the instructions become usable, the simd types still error Unsupported Wasm Type when they appear in function signatures. I was able to reproduce the logic in to_abi_operand from the x64 implementation to use vector registers and larger stack slots for V128. This removes the error from the included .wat files, and flips a number of tests that should have flipped from the simd instructions that have been implemented so far.

@macovedj
macovedj requested review from a team as code owners July 23, 2026 19:01
@macovedj
macovedj requested review from cfallin and removed request for a team July 23, 2026 19:01
@cfallin
cfallin added this pull request to the merge queue Jul 23, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 23, 2026
@alexcrichton
alexcrichton added this pull request to the merge queue Jul 23, 2026
@github-actions github-actions Bot added the winch Winch issues or pull requests label Jul 23, 2026
@github-actions

Copy link
Copy Markdown

Subscribe to Label Action

cc @saulecabrera

Details This issue or pull request has been labeled: "winch"

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

  • saulecabrera: winch

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

Learn more.

@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 23, 2026
@alexcrichton

Copy link
Copy Markdown
Member

ok that one ^ seems legitimate (unlike so many others 😢)

@macovedj
macovedj force-pushed the winch-aarch64-v128-abi branch from 2c8ce8f to e31d876 Compare July 23, 2026 22:18
@alexcrichton
alexcrichton added this pull request to the merge queue Jul 23, 2026
Merged via the queue into bytecodealliance:main with commit 74d98bc Jul 24, 2026
53 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

winch Winch issues or pull requests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants