Re: [PATCH v2 1/3] RISC-V: Remove skip of decl in registered_function.

2024-11-04 Thread Jeff Law
On 11/4/24 6:06 PM, Kito Cheng wrote: Hi Jeff: This should work with the second patch in the patch set, but I think that's not ready yet... Ah. I didn't see anything indicating it relied on the 2nd patch for correctness. I'll let you own this since it hits stuff you know far better than

Re: [PATCH v2 1/3] RISC-V: Remove skip of decl in registered_function.

2024-11-04 Thread Kito Cheng
Hi Jeff: This should work with the second patch in the patch set, but I think that's not ready yet... Jeff Law 於 2024年11月5日 週二 08:52 寫道: > > > On 11/1/24 2:19 AM, KuanLin Chen wrote: > > Hi Jeff, > > > > I'm really sorry for the regression failure. > > I missed one patch to fix these issues. >

Re: [PATCH v2 1/3] RISC-V: Remove skip of decl in registered_function.

2024-11-04 Thread Jeff Law
On 11/1/24 2:19 AM, KuanLin Chen wrote: Hi Jeff, I'm really sorry for the regression failure. I missed one patch to fix these issues. Thanks for your review. The GTY skip makes GGC clean the registered functions wrongly in lto. Example: riscv64-unknown-elf-gcc -flto gcc/testsuite/gcc.target

[PATCH v2 1/3] RISC-V: Remove skip of decl in registered_function.

2024-11-01 Thread KuanLin Chen
Hi Jeff, I'm really sorry for the regression failure. I missed one patch to fix these issues. Thanks for your review. The GTY skip makes GGC clean the registered functions wrongly in lto. Example: riscv64-unknown-elf-gcc -flto gcc/testsuite/gcc.target/riscv/rvv/base/bug-10.c -O2 -march=rv64gcv