Hi! On Fri, Nov 17, 2017 at 07:35:05PM -0500, Michael Meissner wrote: > Here is the fixed patch. It fixes the btrunc<mode>2 insn to use the correct > XSRPQI variant for truncf128. I added the float128-hw11.c test as a runtime > test to make sure round, trunc, ceil, and floor return the correct values. > The > machine independent portions are the same. > > Assuming the machine independent versions are approved, can I check in the > PowerPC bits?
The rs6000 parts are okay, with a trivial fix: > --- gcc/testsuite/gcc.target/powerpc/float128-hw11.c (revision 0) > +++ gcc/testsuite/gcc.target/powerpc/float128-hw11.c (revision 0) > @@ -0,0 +1,59 @@ > +/* { dg-do run { target { powerpc*-*-* && lp64 } } } */ > +/* { dg-require-effective-target powerpc_p9vector_ok } */ > +/* { dg-options "-mpower9-vector -O2" } */ > +/* { dg-options "-mvsx -O2" } */ This last line should not be there. Thanks, Segher