alexcrichton opened PR #6945 from alexcrichton:aarch64-scaled-amode
to bytecodealliance:main
:
This commit adds a few cases to
amode
construction on AArch64 for using theRegScaled*
variants ofAMode
. This won't affect wasm due to this only matching the sign-extension happening before the shift, but it should otherwise help non-wasm Cranelift use cases.Closes #6742
<!--
Please make sure you include the following information:
If this work has been discussed elsewhere, please include a link to that
conversation. If it was discussed in an issue, just mention "issue #...".Explain why this change is needed. If the details are in an issue already,
this can be brief.Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.htmlPlease ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->
alexcrichton requested fitzgen for a review on PR #6945.
alexcrichton requested wasmtime-compiler-reviewers for a review on PR #6945.
fitzgen submitted PR review:
r=me with testing request below addressed
fitzgen submitted PR review:
r=me with testing request below addressed
fitzgen created PR review comment:
Can we add some tests for when the order of the shift vs extend is wrong in the filetests? IIRC we had some historical issues with amodes and doing the extend at the wrong time, so it would be good to get some future-proofing here.
alexcrichton submitted PR review.
alexcrichton created PR review comment:
alexcrichton requested cfallin for a review on PR #6945.
cfallin submitted PR review:
Looks good to me as well, thanks! Thorough tests definitely appreciated.
alexcrichton has enabled auto merge for PR #6945.
fitzgen submitted PR review.
fitzgen created PR review comment:
Ah sorry I thought I only saw
shift(extend(..))
and notextend(shift(..))
fitzgen submitted PR review.
fitzgen created PR review comment:
Thanks!
alexcrichton merged PR #6945.
Last updated: Dec 23 2024 at 12:05 UTC