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