[PATCH] rs6000: Add execution tests for mma builtins.

Segher Boessenkool segher@kernel.crashing.org
Mon Jul 6 18:06:47 GMT 2020


Hi!

On Tue, Jun 30, 2020 at 05:32:24PM -0500, Aaron Sawdey via Gcc-patches wrote:
> This patch adds execution tests that use the MMA builtins,
> checks for the right answer, and checks that __builtin_cpu_supports
> and __builtin_cpu_is return sane answers given that the code
> executed correctly.

> Actually the power10_hw test I think requires
> current glibc to pick up the change that lets
> __builtin_cpu_is("power10") work.

The power10_hw test checks if a "pli" insn works, it doesn't use
__builtin_cpu_is.

> 2020-06-30  Rajalakshmi Srinivasaraghavan  <rajis@linux.vnet.ibm.com>
> 	    Aaron Sawdey  <acsawdey@linux.ibm.com>
> 
> gcc/testsuite/
> 	* gcc.target/powerpc/mma-single-test.c: New file.
> 	* gcc.target/powerpc/mma-double-test.c: New file.

> --- /dev/null
> +++ b/gcc/testsuite/gcc.target/powerpc/mma-double-test.c
> @@ -0,0 +1,211 @@
> +/* { dg-do run } */
> +/* { dg-require-effective-target power10_hw } */
> +/* { dg-options "-Wno-psabi -mdejagnu-cpu=power10 -O2" } */

We probably should have separate selectors for MMA sooner rather than
later (but doesn't have to be *now*).

What is -Wno-psabi needed for?

Okay for trunk with maybe that taken out.  Thanks!  Also okay for 10 if
you want a backport?


Segher


More information about the Gcc-patches mailing list