On Thu, Aug 10, 2017 at 08:27:17AM -0500, Will Schmidt wrote:
> A testcase coverage issue and an obvious typo fix.
>
> Mostly obvious,.. OK for trunk?
Yes; one comment:
> diff --git a/gcc/testsuite/gcc.target/powerpc/fold-vec-pack-longlong.c
> b/gcc/testsuite/gcc.target/powerpc/fold-vec-pack-longlong.c
> index d8acc3c..73131bb 100644
> --- a/gcc/testsuite/gcc.target/powerpc/fold-vec-pack-longlong.c
> +++ b/gcc/testsuite/gcc.target/powerpc/fold-vec-pack-longlong.c
> @@ -1,9 +1,9 @@
> /* Verify that overloaded built-ins for vec_pack with long long
> inputs produce the right results. */
>
> -/* { dg-do compile } */
> +/* { dg-do compile { target { powerpc*-*-* && lp64 } } } */
> /* { dg-require-effective-target powerpc_p8vector_ok } */
> /* { dg-options "-mvsx -mpower8-vector -O2" } */
You can just say
/* { dg-do compile { target lp64 } } */
(make sure to test before you commit :-) )
Segher
