On Thu, 2022-10-20 at 12:52 +0100, Alex Bennée wrote:
> We have finished the TB anyway so we can shortcut the other tests by
> checking dc->ex_value first.
>
> Signed-off-by: Alex Bennée <alex.ben...@linaro.org>
> ---
> target/s390x/tcg/translate.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/target/s390x/tcg/translate.c
> b/target/s390x/tcg/translate.c
> index f4de8efe3a..95279e5dc3 100644
> --- a/target/s390x/tcg/translate.c
> +++ b/target/s390x/tcg/translate.c
> @@ -6624,9 +6624,9 @@ static void
> s390x_tr_translate_insn(DisasContextBase *dcbase, CPUState *cs)
>
> dc->base.is_jmp = translate_one(env, dc);
> if (dc->base.is_jmp == DISAS_NEXT) {
> - if (!is_same_page(dcbase, dc->base.pc_next) ||
> - !is_same_page(dcbase, get_next_pc(env, dc, dc-
> >base.pc_next)) ||
> - dc->ex_value) {
> + if (dc->ex_value ||
> + !is_same_page(dcbase, dc->base.pc_next) ||
> + !is_same_page(dcbase, get_next_pc(env, dc, dc-
> >base.pc_next))) {
> dc->base.is_jmp = DISAS_TOO_MANY;
> }
> }
Acked-by: Ilya Leoshkevich <i...@linux.ibm.com>
