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