Hi Jeff:

This should work with the second patch in the patch set, but I think that's
not ready yet...

Jeff Law <jeffreya...@gmail.com> 於 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.
> > 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