diff options
| author | Michael Meissner <meissner@linux.ibm.com> | 2021-06-30 14:54:48 -0400 |
|---|---|---|
| committer | Michael Meissner <meissner@linux.ibm.com> | 2021-06-30 14:54:48 -0400 |
| commit | 730d021e3e4acc7c0031113ec720c82e31d405e5 (patch) | |
| tree | b77b835c7e872bfdcc537a4c36c84114bb1e42da /gcc | |
| parent | c28e1d288ab727de6eb493e1aa2eadf5d5eef3ab (diff) | |
| download | gcc-730d021e3e4acc7c0031113ec720c82e31d405e5.zip gcc-730d021e3e4acc7c0031113ec720c82e31d405e5.tar.gz gcc-730d021e3e4acc7c0031113ec720c82e31d405e5.tar.bz2 | |
Fix IEEE 128-bit min/max test.
This patch fixes the float128-minmax.c test so that it can accommodate the
generation of xsmincqp and xsmaxcqp instructions on power10. I changed the
effective target from 'float128' to 'ppc_float128_hw', since this needs the
IEEE 128-bit float hardware support. Changing to use 'ppc_float128_hw' allows
the 'lp64' test to be dropped. The 'lp64' test originally was needed because
big endian 32-bit code cannot enable the IEEE 128-bit floating point
instructions.
gcc/testsuite/
2021-06-30 Michael Meissner <meissner@linux.ibm.com>
* gcc.target/powerpc/float128-minmax.c: Adjust expected code for
power10.
* lib/target-supports.exp (check_effective_target_has_arch_pwr10):
New.
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/testsuite/gcc.target/powerpc/float128-minmax.c | 11 | ||||
| -rw-r--r-- | gcc/testsuite/lib/target-supports.exp | 10 |
2 files changed, 16 insertions, 5 deletions
diff --git a/gcc/testsuite/gcc.target/powerpc/float128-minmax.c b/gcc/testsuite/gcc.target/powerpc/float128-minmax.c index fe39751..ef8f729 100644 --- a/gcc/testsuite/gcc.target/powerpc/float128-minmax.c +++ b/gcc/testsuite/gcc.target/powerpc/float128-minmax.c @@ -1,6 +1,4 @@ -/* { dg-do compile { target lp64 } } */ -/* { dg-require-effective-target powerpc_p9vector_ok } */ -/* { dg-require-effective-target float128 } */ +/* { dg-require-effective-target ppc_float128_hw } */ /* { dg-options "-mpower9-vector -O2 -ffast-math" } */ #ifndef TYPE @@ -12,5 +10,8 @@ TYPE f128_min (TYPE a, TYPE b) { return __builtin_fminf128 (a, b); } TYPE f128_max (TYPE a, TYPE b) { return __builtin_fmaxf128 (a, b); } -/* { dg-final { scan-assembler {\mxscmpuqp\M} } } */ -/* { dg-final { scan-assembler-not {\mbl\M} } } */ +/* Adjust code power10 which has native min/max instructions. */ +/* { dg-final { scan-assembler-times {\mxscmpuqp\M} 2 { target { ! has_arch_pwr10 } } } } */ +/* { dg-final { scan-assembler-times {\mxsmincqp\M} 1 { target has_arch_pwr10 } } } */ +/* { dg-final { scan-assembler-times {\mxsmaxcqp\M} 1 { target has_arch_pwr10 } } } */ +/* { dg-final { scan-assembler-not {\mbl\M} } } */ diff --git a/gcc/testsuite/lib/target-supports.exp b/gcc/testsuite/lib/target-supports.exp index 7f78c55..789723f 100644 --- a/gcc/testsuite/lib/target-supports.exp +++ b/gcc/testsuite/lib/target-supports.exp @@ -6127,6 +6127,16 @@ proc check_effective_target_has_arch_pwr9 { } { }] } +proc check_effective_target_has_arch_pwr10 { } { + return [check_no_compiler_messages arch_pwr10 assembly { + #ifndef _ARCH_PWR10 + #error does not have power10 support. + #else + /* "has power10 support" */ + #endif + }] +} + # Return 1 if this is a PowerPC target supporting -mcpu=power10. # Limit this to 64-bit linux systems for now until other targets support # power10. |