diff mbox

target/sh4: optimize cross-page and indirect jumps

Message ID 20170607212058.6124-1-aurelien@aurel32.net (mailing list archive)
State New, archived
Headers show

Commit Message

Aurelien Jarno June 7, 2017, 9:20 p.m. UTC 
Instead of unconditionally exiting to the exec loop for indirect jumps
or cross-page direct jumps, use the lookup_and_goto_ptr helper to jump
to the target if it is valid.

Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
---
 target/sh4/translate.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Richard Henderson June 9, 2017, 5:39 a.m. UTC | #1 
On 06/07/2017 02:20 PM, Aurelien Jarno wrote:
>           if (ctx->singlestep_enabled)
>               gen_helper_debug(cpu_env);
> -        tcg_gen_exit_tb(0);
> +        tcg_gen_lookup_and_goto_ptr(cpu_pc);

That really should be an else for that if, since helper_debug doesn't return.


r~
diff mbox

Patch

diff --git a/target/sh4/translate.c b/target/sh4/translate.c
index 8bc132b27b..4f20537ef8 100644
--- a/target/sh4/translate.c
+++ b/target/sh4/translate.c
@@ -249,7 +249,7 @@  static void gen_goto_tb(DisasContext *ctx, int n, target_ulong dest)
         tcg_gen_movi_i32(cpu_pc, dest);
         if (ctx->singlestep_enabled)
             gen_helper_debug(cpu_env);
-        tcg_gen_exit_tb(0);
+        tcg_gen_lookup_and_goto_ptr(cpu_pc);
     }
 }
 
@@ -262,7 +262,7 @@  static void gen_jump(DisasContext * ctx)
         tcg_gen_discard_i32(cpu_delayed_pc);
 	if (ctx->singlestep_enabled)
             gen_helper_debug(cpu_env);
-	tcg_gen_exit_tb(0);
+        tcg_gen_lookup_and_goto_ptr(cpu_pc);
     } else {
 	gen_goto_tb(ctx, 0, ctx->delayed_pc);
     }