summaryrefslogtreecommitdiffstats
path: root/target/riscv
diff options
context:
space:
mode:
authorRichard Henderson2021-06-21 01:34:21 +0200
committerRichard Henderson2021-07-09 18:42:28 +0200
commitc54d50c1d6d0607ba3b1f803a9284e4b55124522 (patch)
treed338c3e7618ef84492a4503d1eaf0a0533da40a8 /target/riscv
parenttarget/ppc: Use translator_use_goto_tb (diff)
downloadqemu-c54d50c1d6d0607ba3b1f803a9284e4b55124522.tar.gz
qemu-c54d50c1d6d0607ba3b1f803a9284e4b55124522.tar.xz
qemu-c54d50c1d6d0607ba3b1f803a9284e4b55124522.zip
target/riscv: Use translator_use_goto_tb
Just use translator_use_goto_tb directly at the one call site, rather than maintaining a local wrapper. Reviewed-by: Alistair Francis <alistair.francis@wdc.com> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Diffstat (limited to 'target/riscv')
-rw-r--r--target/riscv/translate.c20
1 files changed, 1 insertions, 19 deletions
diff --git a/target/riscv/translate.c b/target/riscv/translate.c
index 62a7d7e4c7..deda0c8a44 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);