commit 611a1684d0b46bae8fb1cfa15db28c0a5cb64cf5
author Richard Henderson <richard.henderson@linaro.org> Wed Jun 28 09:11:56 2023 +0200
committer Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk> Wed Jun 28 10:53:02 2023 +0100
tree 9ec5854f992fd84d834958877e1cf3fea1c0fc68
parent f67ccb2f77190e597562b2041b36ee7d9f6f302b

target/sparc: Fix npc comparison in sparc_tr_insn_start

During translation, npc == address, DYNAMIC_PC, or JUMP_PC.
It is only the encoding between here and sparc_restore_state_to_opc
that considers JUMP_PC to be a bit within a larger value.

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20230628071202.230991-3-richard.henderson@linaro.org>
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>

target/sparc/translate.c

diff --git a/target/sparc/translate.c b/target/sparc/translate.c
index 28d4cdb..eec6f9c 100644
--- a/target/sparc/translate.c
+++ b/target/sparc/translate.c
@@ -5594,7 +5594,7 @@
 {
     DisasContext *dc = container_of(dcbase, DisasContext, base);
 
-    if (dc->npc & JUMP_PC) {
+    if (dc->npc == JUMP_PC) {
         assert(dc->jump_pc[1] == dc->pc + 4);
         tcg_gen_insn_start(dc->pc, dc->jump_pc[0] | JUMP_PC);
     } else {