On 08/10/2018 03:25 PM, Laurent Vivier wrote: > +++ b/target/sh4/translate.c > @@ -253,7 +253,6 @@ static void gen_goto_tb(DisasContext *ctx, int n, > target_ulong dest) > tcg_gen_lookup_and_goto_ptr(); > } > } > - ctx->base.is_jmp = DISAS_NORETURN; > } >
Looking at the other places gen_goto_tb is used, this doesn't look right. Based on the description, I expected a modification in gen_conditional_jump, much like the one you remove here: > @@ -324,7 +323,6 @@ static void gen_delayed_conditional_jump(DisasContext * > ctx) > gen_jump(ctx); > > gen_set_label(l1); > - ctx->base.is_jmp = DISAS_NEXT; > return; > } > r~