[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 18/26] target/riscv: Use translator_use_goto_tb
From: |
Alistair Francis |
Subject: |
Re: [PATCH 18/26] target/riscv: Use translator_use_goto_tb |
Date: |
Tue, 22 Jun 2021 08:22:29 +1000 |
On Mon, Jun 21, 2021 at 11:47 AM Richard Henderson
<richard.henderson@linaro.org> wrote:
>
> Just use translator_use_goto_tb directly at the one call site,
> rather than maintaining a local wrapper.
>
> Cc: qemu-riscv@nongnu.org
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Alistair
> ---
> target/riscv/translate.c | 20 +-------------------
> 1 file changed, 1 insertion(+), 19 deletions(-)
>
> diff --git a/target/riscv/translate.c b/target/riscv/translate.c
> index c6e8739614..ecd3764338 100644
> --- a/target/riscv/translate.c
> +++ b/target/riscv/translate.c
> @@ -168,29 +168,11 @@ static void gen_exception_inst_addr_mis(DisasContext
> *ctx)
> generate_exception_mtval(ctx, RISCV_EXCP_INST_ADDR_MIS);
> }
>
> -static inline bool use_goto_tb(DisasContext *ctx, target_ulong dest)
> -{
> - if (unlikely(ctx->base.singlestep_enabled)) {
> - return false;
> - }
> -
> -#ifndef CONFIG_USER_ONLY
> - return (ctx->base.tb->pc & TARGET_PAGE_MASK) == (dest &
> TARGET_PAGE_MASK);
> -#else
> - return true;
> -#endif
> -}
> -
> static void gen_goto_tb(DisasContext *ctx, int n, target_ulong dest)
> {
> - if (use_goto_tb(ctx, dest)) {
> - /* chaining is only allowed when the jump is to the same page */
> + if (translator_use_goto_tb(&ctx->base, dest)) {
> tcg_gen_goto_tb(n);
> tcg_gen_movi_tl(cpu_pc, dest);
> -
> - /* No need to check for single stepping here as use_goto_tb() will
> - * return false in case of single stepping.
> - */
> tcg_gen_exit_tb(ctx->base.tb, n);
> } else {
> tcg_gen_movi_tl(cpu_pc, dest);
> --
> 2.25.1
>
>