[PATCH] PR 85075, Fix PowerPC __float182/__ibm128 types and mangling

[Michael Meissner]

On Tue, Apr 17, 2018 at 12:29:31PM -0400, Michael Meissner wrote:
> On Tue, Apr 17, 2018 at 09:57:52AM -0500, Segher Boessenkool wrote:
> > On Mon, Apr 16, 2018 at 03:22:16PM -0400, Michael Meissner wrote:
> > > Here is the PR 85075 patch without the rs6000.md bits:
> > 
> > This fails on powerpc64-linux bootstrap (w/ --with-cpu=power7 if that
> > matters):
> > 
> > /home/segher/src/gcc/libgcc/config/rs6000/ibm-ldouble.c: In function '__gcc_qadd
> > ':
> > /home/segher/src/gcc/libgcc/config/rs6000/ibm-ldouble.c:157:1: error: insn does 
> > not satisfy its constraints:
> >  }
> >  ^
> > (insn 144 197 198 15 (set (reg:TF 32 0 [orig:143 _47 ] [143])
> >         (unspec:TF [
> >                 (reg:DF 33 1 [orig:143 _47+8 ] [143])
> >                 (reg/v:DF 36 4 [orig:139 xl ] [139])
> >             ] UNSPEC_PACK_128BIT)) "/home/segher/src/gcc/libgcc/config/rs6000/ib
> > m-ldouble.c":106 1003 {packtf}
> >      (nil))
> > during RTL pass: reload
> 
> Yes, you need the patch for 85358 applied (which you've approved, but I haven't
> checked in yet).  That in fact is why I needed to patch rs6000.md in the
> original patch.

Whoops, it is 85424, not 85835.

And note, you will get a lot of warnings when building libstc++-v3 with the PR
85075 patch, due to the new warning that __float128 changed the name mangling.
I probably should look to see if there is a way to add -Wno-psabi to the
libstdc++-v3 flags on the PowerPC.

-- 
Michael Meissner, IBM
IBM, M/S 2506R, 550 King Street, Littleton, MA 01460-6245, USA
email: meissner@linux.vnet.ibm.com, phone: +1 (978) 899-4797