Hi!

On Wed, Jan 31, 2018 at 01:50:37PM -0600, Will Schmidt wrote:
>   The vec-cmpne-long.c tests targets -mcpu=power8, but always dg-do-run, even
> on P7 (or earlier) systems.  Add a dg-do run requirement for p8vector_hw to
> prevent illegal instruction errors.
> (noticed during review of test results from a P7).
>     
> Sniff tests look good.
>     
> OK for trunk?

Yes please, thanks!  Some trivialities:

> -/* { dg-do run { target { powerpc64*-*-* } } } */
> +/* { dg-do run { target { powerpc64*-*-* && { p8vector_hw } } } } */

Should this be powerpc*-*-* instead?  Which is already implied of course,
this is on gcc.target/powerpc.  And you don't need braces around
p8vector_hw if you put it first.


Segher

Reply via email to