diff options
author | Zack Buhman <zack@buhman.org> | 2024-04-07 23:07:05 +0800 |
---|---|---|
committer | Richard Henderson <richard.henderson@linaro.org> | 2024-04-09 07:43:31 -1000 |
commit | b754cb2dcde26a7bc8a9d17bb6900a0ac0dd38e2 (patch) | |
tree | c83ffe9a595d51b31731beac97223eccefdf9005 /target/sh4 | |
parent | 7227c0cd506eaab5b1d89d15832cac7e05ecb412 (diff) | |
download | qemu-b754cb2dcde26a7bc8a9d17bb6900a0ac0dd38e2.zip qemu-b754cb2dcde26a7bc8a9d17bb6900a0ac0dd38e2.tar.gz qemu-b754cb2dcde26a7bc8a9d17bb6900a0ac0dd38e2.tar.bz2 |
target/sh4: add missing CHECK_NOT_DELAY_SLOT
CHECK_NOT_DELAY_SLOT is correctly applied to the branch-related
instructions, but not to the PC-relative mov* instructions.
I verified the existence of an illegal slot exception on a SH7091 when
any of these instructions are attempted inside a delay slot.
This also matches the behavior described in the SH-4 ISA manual.
Signed-off-by: Zack Buhman <zack@buhman.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20240407150705.5965-1-zack@buhman.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Reviewd-by: Yoshinori Sato <ysato@users.sourceforge.jp>
Diffstat (limited to 'target/sh4')
-rw-r--r-- | target/sh4/translate.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/target/sh4/translate.c b/target/sh4/translate.c index 6643c14..ebb6c90 100644 --- a/target/sh4/translate.c +++ b/target/sh4/translate.c @@ -523,6 +523,7 @@ static void _decode_opc(DisasContext * ctx) tcg_gen_movi_i32(REG(B11_8), B7_0s); return; case 0x9000: /* mov.w @(disp,PC),Rn */ + CHECK_NOT_DELAY_SLOT { TCGv addr = tcg_constant_i32(ctx->base.pc_next + 4 + B7_0 * 2); tcg_gen_qemu_ld_i32(REG(B11_8), addr, ctx->memidx, @@ -530,6 +531,7 @@ static void _decode_opc(DisasContext * ctx) } return; case 0xd000: /* mov.l @(disp,PC),Rn */ + CHECK_NOT_DELAY_SLOT { TCGv addr = tcg_constant_i32((ctx->base.pc_next + 4 + B7_0 * 4) & ~3); tcg_gen_qemu_ld_i32(REG(B11_8), addr, ctx->memidx, @@ -1236,6 +1238,7 @@ static void _decode_opc(DisasContext * ctx) } return; case 0xc700: /* mova @(disp,PC),R0 */ + CHECK_NOT_DELAY_SLOT tcg_gen_movi_i32(REG(0), ((ctx->base.pc_next & 0xfffffffc) + 4 + B7_0 * 4) & ~3); return; |