diff mbox series

ppc: testsuite: skip pr60203 on no ldbl128

Message ID oree2318do.fsf@lxoliva.fsfla.org
State New
Headers
Series ppc: testsuite: skip pr60203 on no ldbl128 |

Commit Message

Alexandre Oliva April 11, 2022, 11:57 p.m. UTC 
  If neither 128-bit long double format is available, skip pr60203.c.

Tested with gcc-11 targeting ppc64-vx7r2, with neither long double
format enabled.  Ok to install?


for  gcc/testsuite/ChangeLog

	* gcc.target/powerpc/pr60203.c: Skip on no 128-bit long double.
---
 gcc/testsuite/gcc.target/powerpc/pr60203.c |    1 +
 1 file changed, 1 insertion(+)

Comments

Segher Boessenkool April 12, 2022, 6:57 p.m. UTC | #1 
Hi!

On Mon, Apr 11, 2022 at 08:57:07PM -0300, Alexandre Oliva wrote:
> If neither 128-bit long double format is available, skip pr60203.c.
> 
> Tested with gcc-11 targeting ppc64-vx7r2, with neither long double
> format enabled.  Ok to install?

Can you use check_effective_target_longdouble128 instead?
/* { dg-require-effective-target longdouble128 } */

Okay for trunk, preferably like that.  Thanks!


Segher
Alexandre Oliva April 13, 2022, 1:16 a.m. UTC | #2 
On Apr 12, 2022, Segher Boessenkool <segher@kernel.crashing.org> wrote:

> Hi!
> On Mon, Apr 11, 2022 at 08:57:07PM -0300, Alexandre Oliva wrote:
>> If neither 128-bit long double format is available, skip pr60203.c.
>> 
>> Tested with gcc-11 targeting ppc64-vx7r2, with neither long double
>> format enabled.  Ok to install?

> Can you use check_effective_target_longdouble128 instead?
> /* { dg-require-effective-target longdouble128 } */

> Okay for trunk, preferably like that.  Thanks!

Thanks, here's what I'm installing

ppc: testsuite: skip pr60203 on no ldbl128

If neither 128-bit long double format is available, skip pr60203.c.


for  gcc/testsuite/ChangeLog

	* gcc.target/powerpc/pr60203.c: Skip on no 128-bit long double.
---
 gcc/testsuite/gcc.target/powerpc/pr60203.c |    1 +
 1 file changed, 1 insertion(+)

diff --git a/gcc/testsuite/gcc.target/powerpc/pr60203.c b/gcc/testsuite/gcc.target/powerpc/pr60203.c
index 7ada64a32db45..a5a574a883729 100644
--- a/gcc/testsuite/gcc.target/powerpc/pr60203.c
+++ b/gcc/testsuite/gcc.target/powerpc/pr60203.c
@@ -1,5 +1,6 @@
 /* { dg-do compile { target { powerpc*-*-* && lp64 } } } */
 /* { dg-skip-if "" { powerpc*-*-darwin* } } */
+/* { dg-require-effective-target longdouble128 } */
 /* { dg-require-effective-target powerpc_p8vector_ok } */
 /* { dg-options "-mdejagnu-cpu=power8 -O3" } */
diff mbox series

Patch

diff --git a/gcc/testsuite/gcc.target/powerpc/pr60203.c b/gcc/testsuite/gcc.target/powerpc/pr60203.c
index 7ada64a32db45..53f58c8e05a41 100644
--- a/gcc/testsuite/gcc.target/powerpc/pr60203.c
+++ b/gcc/testsuite/gcc.target/powerpc/pr60203.c
@@ -1,5 +1,6 @@ 
 /* { dg-do compile { target { powerpc*-*-* && lp64 } } } */
 /* { dg-skip-if "" { powerpc*-*-darwin* } } */
+/* { dg-skip-if "no 128-bit long double" { ! { long_double_ibm128 || long_double_ieee128 } } } */
 /* { dg-require-effective-target powerpc_p8vector_ok } */
 /* { dg-options "-mdejagnu-cpu=power8 -O3" } */