[Valgrind-developers] [valgrind] none/tests/riscv64/testinst.h: Use lla instead of la in JMP_RANGE

[Mark W. <ma...@so...>]

https://sourceware.org/git/gitweb.cgi?p=valgrind.git;h=4091830aa547ac07c50fffa80de2436a82ac017f

commit 4091830aa547ac07c50fffa80de2436a82ac017f
Author: Mark Wielaard <ma...@kl...>
Date:   Thu Apr 24 03:11:26 2025 +0200

    none/tests/riscv64/testinst.h: Use lla instead of la in JMP_RANGE
    
    la was fine when using binutils ld >= 2.43. But older (2.41 and 2.42)
    binutils ld produced:
    
    none/tests/riscv64/integer.c:81:(.text+0x22cd8): dangerous relocation:
    The addend isn't allowed for R_RISCV_GOT_HI20
    
    Using lla makes integer and compressed link with all binutils ld
    versions.

Diff:
---
 none/tests/riscv64/testinst.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/none/tests/riscv64/testinst.h b/none/tests/riscv64/testinst.h
index 2f087d8b71..7f577edc35 100644
--- a/none/tests/riscv64/testinst.h
+++ b/none/tests/riscv64/testinst.h
@@ -403,7 +403,7 @@ static void show_block_diff(unsigned char* block1,
          ".endif;"                                                             \
          ".if \"" #rs1 "\" != \"unused\";"                                     \
          "sd " #rs1 ", 48(%[w]);"      /* Spill rs1. */                        \
-         "la " #rs1 ", " rs1_val ";"   /* Load the first input. */             \
+         "lla " #rs1 ", " rs1_val ";"   /* Load the first input. */            \
          ".endif;"                                                             \
          ".if \"" #rs2 "\" != \"unused\";"                                     \
          "sd " #rs2 ", 56(%[w]);"      /* Spill rs2. */                        \