diff options
| author | 2025-11-15 18:54:21 +0100 | |
|---|---|---|
| committer | 2025-11-15 18:54:21 +0100 | |
| commit | 1910d875cfcad320cbb87c5e8c846d5c53846a1a (patch) | |
| tree | 0c92eeb982f9894ed5701c4713e341f350945f88 /amd64 | |
| parent | 650de3195a9631c9f2c777df5ecf1c6ce26c0430 (diff) | |
isel: don't incorrectly clamp constant lhs of shift operation
Diffstat (limited to 'amd64')
| -rw-r--r-- | amd64/isel.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/amd64/isel.c b/amd64/isel.c index 6635abc..0ed50d9 100644 --- a/amd64/isel.c +++ b/amd64/isel.c @@ -56,10 +56,10 @@ fixarg(union ref *r, struct instr *ins, struct block *blk, int *curi) if (r->t == RXCON) { struct xcon *con = &conht[r->i]; - if (in_range(op, Oshl, Oslr)) { + if (in_range(op, Oshl, Oslr) && r == &ins->r) { sh = con->i; goto ShiftImm; - } else if (in_range(op, Oadd, Osub) && con->i == 2147483648) { + } else if (in_range(op, Oadd, Osub) && con->i == 2147483648 && r == &ins->r) { /* add X, INT32MAX+1 -> sub X, INT32MIN */ ins->op = Oadd + (op == Oadd); *r = mkintcon(KI4, -2147483648); @@ -89,10 +89,10 @@ fixarg(union ref *r, struct instr *ins, struct block *blk, int *curi) *r = insertinstr(blk, (*curi)++, mkinstr(Ocopy, KPTR, mkaddr((struct addr){*r}))); } else if (in_range(op, Odiv, Ourem) && kisint(ins->cls)) goto DivImm; - } else if (r->t == RICON && in_range(op, Odiv, Ourem) && kisint(ins->cls)) { + } else if (r->t == RICON && in_range(op, Odiv, Ourem) && kisint(ins->cls) && r == &ins->r) { DivImm: /* there is no division by immediate, must be copied to a register */ *r = insertinstr(blk, (*curi)++, mkinstr(Ocopy, ins->cls, *r)); - } else if (r->t == RICON && in_range(op, Oshl, Oslr)) { + } else if (r->t == RICON && in_range(op, Oshl, Oslr) && r == &ins->r) { sh = r->i; ShiftImm: /* shift immediate is always 8bit */ *r = mkref(RICON, sh & 255); |