diff options
author | Richard Henderson <rth@twiddle.net> | 2015-10-13 22:07:49 +0000 |
---|---|---|
committer | Richard Henderson <rth@twiddle.net> | 2015-10-28 10:57:16 -0700 |
commit | 522a0d4e3c0d397ffb45ec400d8cbd426dad9d17 (patch) | |
tree | 47bf23d369e201fe8e982097a5fdd5437f88bdfa /target-xtensa | |
parent | 496fedddce9a575111df4f912fb9e361037531ed (diff) | |
download | hqemu-522a0d4e3c0d397ffb45ec400d8cbd426dad9d17.zip hqemu-522a0d4e3c0d397ffb45ec400d8cbd426dad9d17.tar.gz |
target-*: Advance pc after recognizing a breakpoint
Some targets already had this within their logic, but make sure
it's present for all targets.
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Richard Henderson <rth@twiddle.net>
Diffstat (limited to 'target-xtensa')
-rw-r--r-- | target-xtensa/translate.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/target-xtensa/translate.c b/target-xtensa/translate.c index aa0c527..06b0163 100644 --- a/target-xtensa/translate.c +++ b/target-xtensa/translate.c @@ -3088,6 +3088,11 @@ void gen_intermediate_code(CPUXtensaState *env, TranslationBlock *tb) tcg_gen_movi_i32(cpu_pc, dc.pc); gen_exception(&dc, EXCP_DEBUG); dc.is_jmp = DISAS_UPDATE; + /* The address covered by the breakpoint must be included in + [tb->pc, tb->pc + tb->size) in order to for it to be + properly cleared -- thus we increment the PC here so that + the logic setting tb->size below does the right thing. */ + dc.pc += 2; break; } |