Re: [PATCH] testsuite: btf: Fix btf-datasec-1.c for RISC-V

2022-09-09 Thread Mike Stump via Gcc-patches
On May 10, 2022, at 6:31 PM, Kito Cheng via Gcc-patches 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. Yes. Usual and customary for ports.

Re: [PATCH] testsuite: btf: Fix btf-datasec-1.c for RISC-V

2022-05-13 Thread Palmer Dabbelt
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.

Re: [PATCH] testsuite: btf: Fix btf-datasec-1.c for RISC-V

2022-05-10 Thread Kito Cheng via Gcc-patches
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. On Wed, May 4, 2022 at 8:27 AM Palmer Dabbelt wrote: > > Similar to 37e65643d3e ("testsuite/101269: fix testcase when used with > -m32"), RISC-V needs to be told not to put symbols

[PATCH] testsuite: btf: Fix btf-datasec-1.c for RISC-V

2022-05-03 Thread Palmer Dabbelt
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. ---