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 <ra...@linux.vnet.ibm.com>
> Aaron Sawdey <acsaw...@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
