On 07/10/2017 10:27 PM, Alex Bennée wrote:

Richard Henderson <r...@twiddle.net> writes:

On 07/10/2017 09:21 AM, Alex Bennée wrote:
-        s->is_jmp = DISAS_UPDATE;
+        gen_a64_set_pc_im(s->pc);
+        s->is_jmp = DISAS_JUMP;

Better would be gen_goto_tb.  The destination is known, so there's no
need to go through lookup_and_goto_ptr.  You still get the icount_decr
check at the start of the linked TB, which is what you're looking for.

Interesting that a64's gen_goto_tb sets is_jmp, but a32 does not...

Hmm the only caller that is not already in the exit path sets it. Maybe
I should push the s->is_jmp to the a32 gen_goto_tb? I can then do the
same in both.

Sounds reasonable.


r~

Reply via email to