Wasmtime GitHub notifications bot (Sep 16 2024 at 19:07): Wasmtime GitHub notifications bot (Sep 16 2024 at 19:07):bjorn3 opened PR #9258 from bjorn3:abi_cleanup2 to bytecodealliance:main:
Unlike x86_64 and some other architectures where large structs are passed at fixed stack offsets, necessitating special Cranelift support, arm64, riscv64 and s390x pass them using regular pointer arguments that can trivially be implemented by frontends.
The current implementation of StructArgument on arm64 and riscv64 uses fixed stack offsets just like on x86_64, which is incorrect as per the ABI documentation. In other words any correct frontend already had to avoid StructArgument on arm64 and riscv64 anyway. For s390x it is correctly implemented but unnecessary and taints even non-s390x specific code.
Wasmtime GitHub notifications bot (Sep 16 2024 at 19:07):bjorn3 requested wasmtime-compiler-reviewers for a review on PR #9258.
Wasmtime GitHub notifications bot (Sep 16 2024 at 19:07):bjorn3 requested cfallin for a review on PR #9258.
Wasmtime GitHub notifications bot (Sep 16 2024 at 19:11):All cg_clif tests pass on arm64.
Wasmtime GitHub notifications bot (Sep 16 2024 at 19:18):cfallin submitted PR review:
Looks reasonable to me -- thanks for this simplification!
Wasmtime GitHub notifications bot (Sep 16 2024 at 19:18):cfallin has enabled auto merge for PR #9258.
Wasmtime GitHub notifications bot (Sep 16 2024 at 19:37):cfallin merged PR #9258.
Last updated: Nov 22 2024 at 16:03 UTC