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/riscv/rvv/base/bug-10.c
-O2 -march=rv64gcv

In file included from bug-10.c:2: internal compiler error: Segmentation fault

gcc/ChangeLog:

         * config/riscv/riscv-vector-builtins.cc (registered_function):
         Remove skip at decl.

gcc/testsuite/ChangeLog:

         * gcc.target/riscv/rvv/base/bug-10.c: New test.
My point is removing the "skip" is clearly wrong as it causes all kinds of regressions.

This can't be integrated as-is.


jeff

Reply via email to