On Tue, Jan 12, 2016 at 06:48:58PM -0500, David Edelsohn wrote: > On Tue, Jan 12, 2016 at 6:47 PM, Joseph Myers <jos...@codesourcery.com> wrote: > > On Tue, 12 Jan 2016, Michael Meissner wrote: > > > >> On Tue, Jan 12, 2016 at 12:18:55AM +0000, Joseph Myers wrote: > >> > On Mon, 11 Jan 2016, Michael Meissner wrote: > >> > > >> > > I fixed the #ifdef to use __NO_FPRS__ (thanks for the heads up on > >> > > that). I > >> > > also believe I fixed the various formatting issues. These two patches > >> > > build on > >> > > a big endian power7 host and little endian power8 host with no > >> > > regressions in > >> > > the testsuite (the gcc patch is included here, but it hasn't changed > >> > > since the > >> > > previous version of this patch). Are they ok to be checked in? > >> > > >> > Are you sure you sent the right patch version? I don't see those fixes > >> > in > >> > this one. > >> > >> You are right. I did not update the patches from the changes I had made > >> in the > >> branch. > > > > I have no further comments on the patch. > > If Joseph is satisfied, it's okay with me. > > Thanks, David
Because of the failures in my patch and the fact that I will be offline for the next four days, I reverted my IEEE 128-bit libgcc patches. I'm aware of the following breakages: 1) If you build the compiler without --enable-gnu-indirect-function, the compiler is unable to set up the ifunc resolvers to use the IEEE 128-bit hardware that is in ISA 3.0. For this one, I will add code in the configure test to see if we can do IFUNC as well as generate ISA 3.0 instructions. 2) The patch to gcc to add the __ibm128 unpack/pack function breaks the bootstrap on AIX. I will fix both problems and re-submit them on Tuesday when I return. -- 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
