diff mbox series

[RS6000] Unsupported test options for -m32

Message ID 20201025112129.GD15956@bubble.grove.modra.org
State New
Headers show
Series [RS6000] Unsupported test options for -m32 | expand

Commit Message

Alan Modra Oct. 25, 2020, 11:21 a.m. UTC
FAIL: gcc.target/powerpc/swaps-p8-22.c (test for excess errors)
Excess errors:
cc1: error: '-mcmodel' not supported in this configuration

	* gcc.target/powerpc/swaps-p8-22.c: Disable for -m32.

Comments

Segher Boessenkool Oct. 26, 2020, 6:23 p.m. UTC | #1
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 mbox series

Patch

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.