On Sat, Nov 21, 2020 at 10:46:45AM -0500, David Edelsohn wrote: > 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
Thanks. -- Michael Meissner, IBM IBM, M/S 2506R, 550 King Street, Littleton, MA 01460-6245, USA email: meiss...@linux.ibm.com, phone: +1 (978) 899-4797