afonso360 opened PR #5129 from ineg
to main
:
:wave: Hey,
This PR implements
ineg.i128
for all backends, except s390x which already supports it.Fixes #5108
Fixes #5105
afonso360 updated PR #5129 from ineg
to main
.
cfallin submitted PR review.
cfallin created PR review comment:
We unfortunately can't do this (right now) because the zero reg is not allocatable (and if it were, we are also working on removing "pinned vregs", where a vreg refers to a specific physical reg always); @elliottt is working on using fixed-reg operands to allow this in a better way. For now, can we use
(imm $I64 0)
for zeroes (and then we can apply the "use zero reg for a zero" optimization in one place later)?
cfallin submitted PR review.
cfallin created PR review comment:
Likewise here as above
afonso360 updated PR #5129 from ineg
to main
.
cfallin submitted PR review.
afonso360 updated PR #5129 from ineg
to main
.
cfallin merged PR #5129.
Last updated: Dec 23 2024 at 12:05 UTC