Hi, on 2023/5/19 15:40, Ajit Agarwal via Gcc-patches wrote: > Hello All: > > Update powerpc tests for both le and be endian with extra removal of zero > extension and sign extension. > with default ree pass for rs6000 target.
Nice! > > Bootstrapped and regtested on powerpc64-linux-gnu. > > Thanks & Regards > Ajit > > rs6000: Update powerpc test fold-vec-extract-int.p8.c May be better to start with "testsuite:" or "testsuite, rs6000:". > > Update powerpc tests with extra zero_extend removal with default ree pass. > > 2023-05-19 Ajit Kumar Agarwal <aagar...@linux.ibm.com> Since there is an upstream bug tracking this, could you add one PR marker like: PR testsuite/109880 OK for trunk with the PR marker added, thanks! BR, Kewen > > gcc/testsuite/ChangeLog: > > * gcc.target/powerpc/fold-vec-extract-int.p8.c: Update test. > --- > gcc/testsuite/gcc.target/powerpc/fold-vec-extract-int.p8.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/gcc/testsuite/gcc.target/powerpc/fold-vec-extract-int.p8.c > b/gcc/testsuite/gcc.target/powerpc/fold-vec-extract-int.p8.c > index 75eaf25943b..f5f953320d8 100644 > --- a/gcc/testsuite/gcc.target/powerpc/fold-vec-extract-int.p8.c > +++ b/gcc/testsuite/gcc.target/powerpc/fold-vec-extract-int.p8.c > @@ -13,8 +13,8 @@ > > /* { dg-final { scan-assembler-times {\mvspltw\M} 3 { target lp64 } } } */ > /* { dg-final { scan-assembler-times {\mmfvsrwz\M} 3 { target lp64 } } } */ > -/* { dg-final { scan-assembler-times {\mrldicl\M} 7 { target { le } } } } */ > -/* { dg-final { scan-assembler-times {\mrldicl\M} 4 { target { lp64 && be } > } } } */ > +/* { dg-final { scan-assembler-times {\mrldicl\M} 5 { target { le } } } } */ > +/* { dg-final { scan-assembler-times {\mrldicl\M} 2 { target { lp64 && be } > } } } */ > /* { dg-final { scan-assembler-times {\msubfic\M} 3 { target { le } } } } */ > /* { dg-final { scan-assembler-times {\msldi\M} 3 { target lp64 } } } */ > /* { dg-final { scan-assembler-times {\mmtvsrd\M} 3 { target lp64 } } } */