Message ID | 20201025112129.GD15956@bubble.grove.modra.org |
---|---|
State | New |
Headers | show |
Series | [RS6000] Unsupported test options for -m32 | expand |
On Sun, Oct 25, 2020 at 09:51:29PM +1030, Alan Modra wrote: > FAIL: gcc.target/powerpc/swaps-p8-22.c (test for excess errors) > Excess errors: > cc1: error: '-mcmodel' not supported in this configuration This is because your build is not biarch. We really should not allow such configurations, they are extra work for us, for no added value. > diff --git a/gcc/testsuite/gcc.target/powerpc/swaps-p8-22.c b/gcc/testsuite/gcc.target/powerpc/swaps-p8-22.c > index 83f6ab3a1c0..bceada41b75 100644 > --- a/gcc/testsuite/gcc.target/powerpc/swaps-p8-22.c > +++ b/gcc/testsuite/gcc.target/powerpc/swaps-p8-22.c > @@ -1,5 +1,5 @@ > /* { dg-do compile } */ > -/* { dg-require-effective-target powerpc_p8vector_ok } */ > +/* { dg-require-effective-target { lp64 && powerpc_p8vector_ok } } */ > /* { dg-options "-O2 -mdejagnu-cpu=power8 -maltivec -mcmodel=large" } */ > > /* The expansion for vector character multiply introduces a vperm operation. Add a comment why we need lp64? (Easiest is if you use two separate dg-require phrases.) With that, okay for trunk. Thanks! Segher
diff --git a/gcc/testsuite/gcc.target/powerpc/swaps-p8-22.c b/gcc/testsuite/gcc.target/powerpc/swaps-p8-22.c index 83f6ab3a1c0..bceada41b75 100644 --- a/gcc/testsuite/gcc.target/powerpc/swaps-p8-22.c +++ b/gcc/testsuite/gcc.target/powerpc/swaps-p8-22.c @@ -1,5 +1,5 @@ /* { dg-do compile } */ -/* { dg-require-effective-target powerpc_p8vector_ok } */ +/* { dg-require-effective-target { lp64 && powerpc_p8vector_ok } } */ /* { dg-options "-O2 -mdejagnu-cpu=power8 -maltivec -mcmodel=large" } */ /* The expansion for vector character multiply introduces a vperm operation.