On Thu, Nov 13, 2025 at 7:27 PM Artemiy Volkov <[email protected]> wrote:
>
> On Thu, Nov 13, 2025 at 09:23:58AM -0800, Andrew Pinski wrote:
> > On Thu, Nov 13, 2025 at 8:56 AM Artemiy Volkov <[email protected]>
> > wrote:
> > >
> > > Introduced in r16-5042-g470411f44f51d9, this testcase fails on
> > > AdvSIMD-less AArch32 configurations, likely as well as on other targets
> > > without vector support; thus, require it via dg-require-effective-target.
> > > Since this testcase includes stdint.h, require that as well.
> > >
> > > Regtested on arm-gnueabihf with
> > > RUNTESTFLAGS=--target_board=unix/-mfpu=vfpv3-d16/-march=armv7-a.
> > >
> > > Pushing as obvious.
> > >
> > > gcc/testsuite/ChangeLog:
> > >
> > > * gcc.dg/tree-ssa/forwprop-43.c: Adjust.
> > > ---
> > > gcc/testsuite/gcc.dg/tree-ssa/forwprop-43.c | 2 ++
> > > 1 file changed, 2 insertions(+)
> > >
> > > diff --git a/gcc/testsuite/gcc.dg/tree-ssa/forwprop-43.c
> > > b/gcc/testsuite/gcc.dg/tree-ssa/forwprop-43.c
> > > index f0f6170648a..bfda376e1f7 100644
> > > --- a/gcc/testsuite/gcc.dg/tree-ssa/forwprop-43.c
> > > +++ b/gcc/testsuite/gcc.dg/tree-ssa/forwprop-43.c
> > > @@ -1,5 +1,7 @@
> > > /* { dg-do compile } */
> > > /* { dg-options "-O2 -fdump-tree-forwprop1" } */
> > > +/* { dg-require-effective-target stdint_types } */
> > > +/* { dg-require-effective-target vect_int } */
> >
> > The requirement of vect_int should NOT be needed as you are scanning
> > forwprop1 and it can't be used outside of the vect/* test
> > infrastructure see r14-8210-g484f48f03cf9a3 on why.
>
> I guess it wasn't that obvious after all...
>
> In this case I think I could just move the test to gcc.dg/vect, would that
> be an acceptable solution?
The testcase should pass for targets w/o vector support. The only
reason it might not pass is when there's target support for the
vector mode used and there's no original permute or the original
permute is supported by the target but the replacement is not.
Plain vect_int isn't the appropriate test in any case.
So I think this needs a more thorough analysis what exactly fails and why.
Richard.
> Thanks,
> Artemiy
>
> >
> > Thanks,
> > Andrew Pinski
> >
> >
> > > /* { dg-additional-options "-fgimple" } */
> > >
> > > #include <stdint.h>
> > > --
> > > 2.43.0
> > >
