Hi Alan,
On Sun, Oct 25, 2020 at 09:50:01PM +1030, Alan Modra wrote:
> All these tests fail with -m32 due to lack of int128 support,
Is there any good reason __int128 is not enabled for rs6000 -m32, btw?
> in some
> cases with what I thought was not the best error message. For example
> vsx_mask-move-runnable.c:34:3: error: unknown type name 'vector'
> is misleading. The problem isn't "vector" but "vector __uint128_t".
Ouch, yes. Do you see a simple way to fix that?
> diff --git a/gcc/testsuite/gcc.target/powerpc/vsx-load-element-extend-char.c
> b/gcc/testsuite/gcc.target/powerpc/vsx-load-element-extend-char.c
> index 0b8cfd610f8..7a7cb77c3a0 100644
> --- a/gcc/testsuite/gcc.target/powerpc/vsx-load-element-extend-char.c
> +++ b/gcc/testsuite/gcc.target/powerpc/vsx-load-element-extend-char.c
> @@ -4,6 +4,7 @@
>
> /* { dg-do compile {target power10_ok} } */
> /* { dg-do run {target power10_hw} } */
> +/* { dg-require-effective-target { int128 } } */
> /* { dg-options "-mdejagnu-cpu=power10 -O3" } */
You might want to write this as {int128}, to keep the same style as the
other statements. Or leave off the braces completely, they aren't
necessary here, int128 is a single word :-)
> diff --git a/gcc/testsuite/gcc.target/powerpc/vsx_mask-count-runnable.c
> b/gcc/testsuite/gcc.target/powerpc/vsx_mask-count-runnable.c
> index 5862517eae9..6ac4ed2173f 100644
> --- a/gcc/testsuite/gcc.target/powerpc/vsx_mask-count-runnable.c
> +++ b/gcc/testsuite/gcc.target/powerpc/vsx_mask-count-runnable.c
> @@ -1,7 +1,7 @@
> /* { dg-do run { target { power10_hw } } } */
> /* { dg-do link { target { ! power10_hw } } } */
> /* { dg-options "-mcpu=power10 -O2" } */
> -/* { dg-require-effective-target power10_ok } */
> +/* { dg-require-effective-target { int128 && power10_ok } } */
Or write it as two require statements, as we do most of the time?
Okay for trunk (with those tweaks if you want). Thanks!
Segher
