From c628f812eba820e11ef83033b38becca1b148dc5 Mon Sep 17 00:00:00 2001 From: Lewis Revill Date: Fri, 22 Oct 2021 11:32:46 +0800 Subject: [PATCH] RISC-V: Added ld testcase for pcgp relaxation. Consider the the pcgp-relax-02 testcase, .text .globl _start _start: .L1: auipc a0, %pcrel_hi(data_a) .L2: auipc a1, %pcrel_hi(data_b) addi a0, a0, %pcrel_lo(.L1) addi a1, a1, %pcrel_lo(.L2) .data .word 0x0 .globl data_a data_a: .word 0x1 .section .rodata .globl data_b data_b: .word 0x2 If the first auipc is deleted, but we are still building the pcgp table (connect the high and low pcrel relocations), then there is an aliasing issue that we need some way to disambiguate which of the two symbols we are targeting. Therefore, Palmer thought of a way to use R_RISCV_DELETE to split this into two phases, so we could resolve the addresses before creating the ambiguities. This patch just add the ld testcase for the above case, in case we have changed something but break this. ld/ * testsuite/ld-riscv-elf/ld-riscv-elf.exp: Renamed pcgp-relax to pcgp-relax-01, and added pcgp-relax-02. * testsuite/ld-riscv-elf/pcgp-relax-01.d: Renmaed from pcgp-relax. * testsuite/ld-riscv-elf/pcgp-relax-01.s: Likewise. * testsuite/ld-riscv-elf/pcgp-relax-02.d: New testcase. * testsuite/ld-riscv-elf/pcgp-relax-02.s: Likewise. --- ld/testsuite/ld-riscv-elf/ld-riscv-elf.exp | 3 ++- .../{pcgp-relax.d => pcgp-relax-01.d} | 2 +- .../{pcgp-relax.s => pcgp-relax-01.s} | 0 ld/testsuite/ld-riscv-elf/pcgp-relax-02.d | 15 +++++++++++++++ ld/testsuite/ld-riscv-elf/pcgp-relax-02.s | 18 ++++++++++++++++++ 5 files changed, 36 insertions(+), 2 deletions(-) rename ld/testsuite/ld-riscv-elf/{pcgp-relax.d => pcgp-relax-01.d} (94%) rename ld/testsuite/ld-riscv-elf/{pcgp-relax.s => pcgp-relax-01.s} (100%) create mode 100644 ld/testsuite/ld-riscv-elf/pcgp-relax-02.d create mode 100644 ld/testsuite/ld-riscv-elf/pcgp-relax-02.s diff --git a/ld/testsuite/ld-riscv-elf/ld-riscv-elf.exp b/ld/testsuite/ld-riscv-elf/ld-riscv-elf.exp index 20ca3bdb195..78a71345bf2 100644 --- a/ld/testsuite/ld-riscv-elf/ld-riscv-elf.exp +++ b/ld/testsuite/ld-riscv-elf/ld-riscv-elf.exp @@ -121,7 +121,8 @@ proc run_relax_twice_test {} { if [istarget "riscv*-*-*"] { run_dump_test "align-small-region" run_dump_test "call-relax" - run_dump_test "pcgp-relax" + run_dump_test "pcgp-relax-01" + run_dump_test "pcgp-relax-02" run_dump_test "c-lui" run_dump_test "c-lui-2" run_dump_test "disas-jalr" diff --git a/ld/testsuite/ld-riscv-elf/pcgp-relax.d b/ld/testsuite/ld-riscv-elf/pcgp-relax-01.d similarity index 94% rename from ld/testsuite/ld-riscv-elf/pcgp-relax.d rename to ld/testsuite/ld-riscv-elf/pcgp-relax-01.d index dae2b62b9a7..f9425678559 100644 --- a/ld/testsuite/ld-riscv-elf/pcgp-relax.d +++ b/ld/testsuite/ld-riscv-elf/pcgp-relax-01.d @@ -1,4 +1,4 @@ -#source: pcgp-relax.s +#source: pcgp-relax-01.s #ld: --relax #objdump: -d -Mno-aliases diff --git a/ld/testsuite/ld-riscv-elf/pcgp-relax.s b/ld/testsuite/ld-riscv-elf/pcgp-relax-01.s similarity index 100% rename from ld/testsuite/ld-riscv-elf/pcgp-relax.s rename to ld/testsuite/ld-riscv-elf/pcgp-relax-01.s diff --git a/ld/testsuite/ld-riscv-elf/pcgp-relax-02.d b/ld/testsuite/ld-riscv-elf/pcgp-relax-02.d new file mode 100644 index 00000000000..c6c73c54265 --- /dev/null +++ b/ld/testsuite/ld-riscv-elf/pcgp-relax-02.d @@ -0,0 +1,15 @@ +#source: pcgp-relax-02.s +#as: +#ld: --relax +#objdump: -d + +.*:[ ]+file format .* + + +Disassembly of section .text: + +[0-9a-f]+ <_start>: +.*:[ ]+[0-9a-f]+[ ]+auipc[ ]+a1.* +.*:[ ]+[0-9a-f]+[ ]+addi[ ]+a0,gp.* +.*:[ ]+[0-9a-f]+[ ]+addi[ ]+a1,a1.* +#pass diff --git a/ld/testsuite/ld-riscv-elf/pcgp-relax-02.s b/ld/testsuite/ld-riscv-elf/pcgp-relax-02.s new file mode 100644 index 00000000000..ca4aa1fa9d2 --- /dev/null +++ b/ld/testsuite/ld-riscv-elf/pcgp-relax-02.s @@ -0,0 +1,18 @@ + .text + .globl _start +_start: +.L1: auipc a0, %pcrel_hi(data_a) +.L2: auipc a1, %pcrel_hi(data_b) + addi a0, a0, %pcrel_lo(.L1) + addi a1, a1, %pcrel_lo(.L2) + + .data + .word 0x0 + .globl data_a +data_a: + .word 0x1 + + .section .rodata + .globl data_b +data_b: + .word 0x2 -- 2.30.2