[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 5/6] target/sh4: Do not singlestep after exceptions
From: |
Aurelien Jarno |
Subject: |
[Qemu-devel] [PULL 5/6] target/sh4: Do not singlestep after exceptions |
Date: |
Mon, 18 Dec 2017 23:40:29 +0100 |
From: Richard Henderson <address@hidden>
If we've already raised an exception (and set NORETURN),
do not emit unreachable code to raise a debug exception.
Note that gen_goto_tb takes single-stepping into account.
Signed-off-by: Richard Henderson <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Aurelien Jarno <address@hidden>
---
target/sh4/translate.c | 32 ++++++++++++++++----------------
1 file changed, 16 insertions(+), 16 deletions(-)
diff --git a/target/sh4/translate.c b/target/sh4/translate.c
index efd1081c57..dd9aaa4e63 100644
--- a/target/sh4/translate.c
+++ b/target/sh4/translate.c
@@ -270,6 +270,7 @@ static void gen_jump(DisasContext * ctx)
} else {
tcg_gen_lookup_and_goto_ptr();
}
+ ctx->bstate = DISAS_NORETURN;
} else {
gen_goto_tb(ctx, 0, ctx->delayed_pc);
}
@@ -2341,24 +2342,23 @@ void gen_intermediate_code(CPUState *cs, struct
TranslationBlock *tb)
ctx.envflags &= ~GUSA_MASK;
}
- if (cs->singlestep_enabled) {
+ switch (ctx.bstate) {
+ case DISAS_STOP:
gen_save_cpu_state(&ctx, true);
- gen_helper_debug(cpu_env);
- } else {
- switch (ctx.bstate) {
- case DISAS_STOP:
- gen_save_cpu_state(&ctx, true);
+ if (cs->singlestep_enabled) {
+ gen_helper_debug(cpu_env);
+ } else {
tcg_gen_exit_tb(0);
- break;
- case DISAS_NEXT:
- gen_save_cpu_state(&ctx, false);
- gen_goto_tb(&ctx, 0, ctx.pc);
- break;
- case DISAS_NORETURN:
- break;
- default:
- g_assert_not_reached();
- }
+ }
+ break;
+ case DISAS_NEXT:
+ gen_save_cpu_state(&ctx, false);
+ gen_goto_tb(&ctx, 0, ctx.pc);
+ break;
+ case DISAS_NORETURN:
+ break;
+ default:
+ g_assert_not_reached();
}
gen_tb_end(tb, num_insns);
--
2.15.1
- [Qemu-devel] [PULL 0/6] Queued target/sh4 patches, Aurelien Jarno, 2017/12/18
- [Qemu-devel] [PULL 1/6] target/sh4: add missing tcg_temp_free() in _decode_opc(), Aurelien Jarno, 2017/12/18
- [Qemu-devel] [PULL 3/6] target/sh4: Use cmpxchg for movco when parallel_cpus, Aurelien Jarno, 2017/12/18
- [Qemu-devel] [PULL 2/6] target/sh4: fix TCG leak during gusa sequence, Aurelien Jarno, 2017/12/18
- [Qemu-devel] [PULL 4/6] target/sh4: Convert to DisasJumpType, Aurelien Jarno, 2017/12/18
- [Qemu-devel] [PULL 6/6] target/sh4: Convert to DisasContextBase, Aurelien Jarno, 2017/12/18
- [Qemu-devel] [PULL 5/6] target/sh4: Do not singlestep after exceptions,
Aurelien Jarno <=
- Re: [Qemu-devel] [PULL 0/6] Queued target/sh4 patches, Peter Maydell, 2017/12/19