@samitouri / QOSamiQemu / commits / 6981e88add

target/arm/tcg/translate-a64.c: use translator_ldl_end instead of arm_ldl_code

Allows to reduce scope of target/arm/tcg/arm_ldst.h to aarch32 only. Suggested-by: Richard Henderson <richard.henderson@linaro.org> Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org> Message-id: 20260407222208.271838-17-pierrick.bouvier@linaro.org Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

Pierrick Bouvier committed Apr 23, 2026 at 10:24 UTC 6981e88addb7476cfa3532b509abfec50f75ebcf
1 file changed +3 -3
target/arm/tcg/translate-a64.c
+3 -3
@@ -26,7 +26,6 @@
26 #include "translate-a64.h"
27 #include "tcg/tcg-op.h"
28 #include "qemu/log.h"
29 -#include "arm_ldst.h"
29 #include "semihosting/semihost.h"
30 #include "cpregs.h"
31
@@ -10801,7 +10800,7 @@ static void aarch64_tr_translate_insn(DisasContextBase *dcbase, CPUState *cpu)
10800 if (pc & 3) {
10801 /*
10802 * PC alignment fault. This has priority over the instruction abort
10804 - * that we would receive from a translation fault via arm_ldl_code.
10803 + * that we would receive from a translation fault via translator_ldl_end.
10804 * This should only be possible after an indirect branch, at the
10805 * start of the TB.
10806 */
@@ -10813,7 +10812,8 @@ static void aarch64_tr_translate_insn(DisasContextBase *dcbase, CPUState *cpu)
10812 }
10813
10814 s->pc_curr = pc;
10816 - insn = arm_ldl_code(env, &s->base, pc, s->sctlr_b);
10815 + /* Code is always little-endian on Aarch64 */
10816 + insn = translator_ldl_end(env, &s->base, pc, MO_LE);
10817 s->insn = insn;
10818 s->base.pc_next = pc + 4;
10819