On Sat, Nov 21, 2020 at 12:37 AM Michael Meissner <meiss...@linux.ibm.com> wrote: > > PowerPC: require IBM long double for pr70117. > > Since the test is explicitly checking for IBM extended double, do not try to > run it when long double is IEEE 128-bit. > > I have tested this patch and the first patch in the series on a little endian > power9 system with 3 compilers that have the 3 different long double types > configured. I verified that this test is listed as unsupported if long double > is not IBM 128-bit, and it passes if long double is IBM 128-bit. > > Can I check this patch into the master branch? After an appropriate soak-in > time, can I back port this patch to the previous open GCC trees? > > gcc/testsuite/ > 2020-11-21 Michael Meissner <meiss...@linux.ibm.com> > > PR target/70117 > * gcc.target/powerpc/pr70117.c: Only run the test if long double > uses the IBM extended double format. > --- > gcc/testsuite/gcc.target/powerpc/pr70117.c | 22 ++++------------------ > 1 file changed, 4 insertions(+), 18 deletions(-) > > diff --git a/gcc/testsuite/gcc.target/powerpc/pr70117.c > b/gcc/testsuite/gcc.target/powerpc/pr70117.c > index 3bbd2c595e0..2077d15afd4 100644 > --- a/gcc/testsuite/gcc.target/powerpc/pr70117.c > +++ b/gcc/testsuite/gcc.target/powerpc/pr70117.c > @@ -1,26 +1,12 @@ > -/* { dg-do run { target { powerpc*-*-linux* powerpc*-*-darwin* > powerpc*-*-aix* rs6000-*-* } } } */ > -/* { dg-options "-std=c99 -mlong-double-128 -O2" } */ > +/* { dg-require-effective-target ppc_long_double_ibm } */ > +/* { dg-options "-std=c99 -O2" } */
Mike, You still need a /* { dg-do run } */ line. But without any target specified. Segher may have other comments. Thanks, David