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: meiss...@linux.vnet.ibm.com, phone: +1 (978) 899-4797