On Fri, May 26, 2017 at 10:32:54AM -0500, Steven Munroe wrote:
>       * gcc.target/powerpc/bmi2-pdep32-1.c []: Add -mcpu=power7 to
>       dg-options.  Change dg-require-effective-target powerpc_vsx_ok
>       to vsx_hw.

Stray "[]"?

> --- gcc/testsuite/gcc.target/powerpc/bmi2-pdep32-1.c  (revision 248468)
> +++ gcc/testsuite/gcc.target/powerpc/bmi2-pdep32-1.c  (working copy)
> @@ -1,7 +1,7 @@
>  /* { dg-do run } */
> -/* { dg-options "-O3" } */
> +/* { dg-options "-O3 -mcpu=power7" } */
>  /* { dg-require-effective-target lp64 } */
> -/* { dg-require-effective-target powerpc_vsx_ok } */
> +/* { dg-require-effective-target vsx_hw } */

Other testcases selecting a -mcpu= also use

/* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } { 
"-mcpu=power7" } } */

Do you really want -mcpu=power7 always?  Or just at least power7?


Segher

Reply via email to