On Tue, 10 May 2022 18:31:37 PDT (-0700), Kito Cheng wrote:
LGTM, that's only added a new option for RISC-V and won't affect all other targets, so I assume I can approve that.
I always forget what the rules are, but someone recently reminded me and yes that's the case. I committed this.
Thanks!
On Wed, May 4, 2022 at 8:27 AM Palmer Dabbelt <pal...@rivosinc.com> wrote:Similar to 37e65643d3e ("testsuite/101269: fix testcase when used with -m32"), RISC-V needs to be told not to put symbols in the sdata/srodata/sbss sections. gcc/testsuite/ChangeLog * debug/btf/btf-datasec-1.c: Don't use small data on RISC-V. --- gcc/testsuite/gcc.dg/debug/btf/btf-datasec-1.c | 1 + 1 file changed, 1 insertion(+) diff --git a/gcc/testsuite/gcc.dg/debug/btf/btf-datasec-1.c b/gcc/testsuite/gcc.dg/debug/btf/btf-datasec-1.c index dbb236bbda1..77df88648c5 100644 --- a/gcc/testsuite/gcc.dg/debug/btf/btf-datasec-1.c +++ b/gcc/testsuite/gcc.dg/debug/btf/btf-datasec-1.c @@ -12,6 +12,7 @@ /* { dg-do compile ) */ /* { dg-options "-O0 -gbtf -dA" } */ /* { dg-options "-O0 -gbtf -dA -msdata=none" { target { { powerpc*-*-* } && ilp32 } } } */ +/* { dg-options "-O0 -gbtf -dA -msmall-data-limit=0" { target { riscv*-*-* } } } */ /* { dg-options "-O0 -gbtf -dA -G0" { target { nios2-*-* } } } */ /* Check for two DATASEC entries with vlen 3, and one with vlen 1. */ -- 2.34.1
