target/arm: Use DISAS_TOO_MANY for ISB and SB
Using gen_goto_tb directly misses the single-step check. Let the branch or debug exception be emitted by arm_tr_tb_stop. Reviewed-by: Peter Maydell <peter.maydell@linaro.org> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
This commit is contained in:
parent
21a7e89ece
commit
73fce314db
|
@ -8904,7 +8904,7 @@ static bool trans_ISB(DisasContext *s, arg_ISB *a)
|
||||||
* self-modifying code correctly and also to take
|
* self-modifying code correctly and also to take
|
||||||
* any pending interrupts immediately.
|
* any pending interrupts immediately.
|
||||||
*/
|
*/
|
||||||
gen_goto_tb(s, 0, s->base.pc_next);
|
s->base.is_jmp = DISAS_TOO_MANY;
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -8918,7 +8918,7 @@ static bool trans_SB(DisasContext *s, arg_SB *a)
|
||||||
* for TCG; MB and end the TB instead.
|
* for TCG; MB and end the TB instead.
|
||||||
*/
|
*/
|
||||||
tcg_gen_mb(TCG_MO_ALL | TCG_BAR_SC);
|
tcg_gen_mb(TCG_MO_ALL | TCG_BAR_SC);
|
||||||
gen_goto_tb(s, 0, s->base.pc_next);
|
s->base.is_jmp = DISAS_TOO_MANY;
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue