saulecabrera opened PR #5461 from x64-imm-mem-mov
to main
:
This change adds support for immediate to memory moves in x64 which are needed by Winch for zeroing local slots.
This change follows the guideline in
isa/x64/inst/emit
and uses other instructions (immediate to register moves) as a base for the test cases.The instruction encoding expectation was derived by assembling each instruction and inspecting the assembly with
objdump
.<!--
Please ensure that the following steps are all taken care of before submitting
the PR.
[ ] This has been discussed in issue #..., or if not, please tell us why
here.[ ] A short description of what this does, why it is needed; if the
description becomes long, the matter should probably be discussed in an issue
first.[ ] This PR contains test cases, if meaningful.
- [ ] A reviewer from the core maintainer team has been assigned for this PR.
If you don't know who could review this, please indicate so. The list of
suggested reviewers on the right can help you.Please ensure all communication adheres to the code of conduct.
-->
saulecabrera requested cfallin for a review on PR #5461.
cfallin created PR review comment:
Can we add
Size16
andSize8
cases here too?
cfallin submitted PR review.
cfallin submitted PR review.
cfallin created PR review comment:
Add a comment here perhaps that we don't need
always_emit_if_8bit_needed()
on the REX flags, because the dst is a memory operand, not a possibly 8-bit register?
saulecabrera updated PR #5461 from x64-imm-mem-mov
to main
.
saulecabrera submitted PR review.
saulecabrera created PR review comment:
Done, thanks!
saulecabrera created PR review comment:
Agreed, done!
saulecabrera submitted PR review.
saulecabrera requested cfallin for a review on PR #5461.
cfallin submitted PR review.
cfallin submitted PR review.
cfallin created PR review comment:
Can we update this comment to include the prefix as well?
saulecabrera updated PR #5461 from x64-imm-mem-mov
to main
.
saulecabrera submitted PR review.
saulecabrera created PR review comment:
Good catch, thanks! Updated!
cfallin submitted PR review.
cfallin has enabled auto merge for PR #5461.
cfallin merged PR #5461.
Last updated: Dec 23 2024 at 13:07 UTC