On Thu, Oct 25, 2018 at 02:07:45PM -0500, Paul Clarke wrote:
> Enable 32bit support for tests of x86-compatibile intrinsics
> 
> Bootstrapped and tested on Linux POWER8 LE, POWER8 BE (64 & 32), and POWER7.
> 
> OK for trunk?

This is fine when the previous patch is in as well.  Thanks!

> 2018-10-25  Paul A. Clarke  <p...@us.ibm.com>
> 
>       * gcc.target/powerpc/mmx-packs.c: Enable 32 bit execution.

>       * gcc.target/powerpc/mmx-packuswb-1.c: Likewise.

(You don't mention the one more interesting change, heh):

> --- a/trunk/gcc/testsuite/gcc.target/powerpc/mmx-packuswb-1.c (revision 
> 265238)
> +++ b/trunk/gcc/testsuite/gcc.target/powerpc/mmx-packuswb-1.c (working copy)
> @@ -1,6 +1,5 @@
>  /* { dg-do run } */
> -/* { dg-options "-O3" } */
> -/* { dg-require-effective-target lp64 } */
> +/* { dg-options "-O3 -mpower8-vector" } */
>  /* { dg-require-effective-target p8vector_hw } */

(Which is just a simple bugfix, so that is fine, but do mention it please).


Segher

Reply via email to