On Mon, Nov 20, 2017 at 11:32:08AM -0600, Segher Boessenkool wrote:
> 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.
Ok. I will also substitute p9vector_hw for powerpc_p9vector_ok, since the
former says we have power9 hardware to run tests, and the later says the
compiler supports compiling for a power9 target. Sorry about that.
--
Michael Meissner, IBM
IBM, M/S 2506R, 550 King Street, Littleton, MA 01460-6245, USA
email: meiss...@linux.vnet.ibm.com, phone: +1 (978) 899-4797
