diff options
| author | Richard Henderson | 2014-05-15 06:09:47 +0200 |
|---|---|---|
| committer | Richard Henderson | 2014-05-15 06:09:47 +0200 |
| commit | a1756896546fccf39eef6f282dcbd2f9292c1286 (patch) | |
| tree | 6cc2c1a9712c9f169e1f6d7db35bf64be0c9e2f6 | |
| parent | tcg: Fix tcg_reg_alloc_mov vs no-op truncation (diff) | |
| download | qemu-a1756896546fccf39eef6f282dcbd2f9292c1286.tar.gz qemu-a1756896546fccf39eef6f282dcbd2f9292c1286.tar.xz qemu-a1756896546fccf39eef6f282dcbd2f9292c1286.zip | |
tcg-s390: Fix off-by-one in wraparound andi
Signed-off-by: Richard Henderson <rth@twiddle.net>
| -rw-r--r-- | tcg/s390/tcg-target.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tcg/s390/tcg-target.c b/tcg/s390/tcg-target.c index ebdd0743cf..0f972ae5df 100644 --- a/tcg/s390/tcg-target.c +++ b/tcg/s390/tcg-target.c @@ -983,8 +983,8 @@ static void tgen_andi(TCGContext *s, TCGType type, TCGReg dest, uint64_t val) int msb, lsb; if ((val & 0x8000000000000001ull) == 0x8000000000000001ull) { /* Achieve wraparound by swapping msb and lsb. */ - msb = 63 - ctz64(~val); - lsb = clz64(~val) + 1; + msb = 64 - ctz64(~val); + lsb = clz64(~val) - 1; } else { msb = clz64(val); lsb = 63 - ctz64(val); |