This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Ping Re: Implement C _FloatN, _FloatNx types [version 3]
- From: Segher Boessenkool <segher at kernel dot crashing dot org>
- To: Joseph Myers <joseph at codesourcery dot com>
- Cc: Bill Schmidt <wschmidt at linux dot vnet dot ibm dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, fortran at gcc dot gnu dot org, jason at redhat dot com, richard dot earnshaw at arm dot com, nickc at redhat dot com, ramana dot radhakrishnan at arm dot com, marcus dot shawcroft at arm dot com, dje dot gcc at gmail dot com, meissner at linux dot vnet dot ibm dot com, murphyp at linux dot vnet dot ibm dot com, bje at gnu dot org
- Date: Mon, 27 Jun 2016 16:35:16 -0500
- Subject: Re: Ping Re: Implement C _FloatN, _FloatNx types [version 3]
- Authentication-results: sourceware.org; auth=none
- References: <alpine dot DEB dot 2 dot 20 dot 1606211202040 dot 4526 at digraph dot polyomino dot org dot uk> <alpine dot DEB dot 2 dot 20 dot 1606211738200 dot 31330 at digraph dot polyomino dot org dot uk> <alpine dot DEB dot 2 dot 20 dot 1606231418120 dot 21240 at digraph dot polyomino dot org dot uk> <alpine dot DEB dot 2 dot 20 dot 1606271720270 dot 7438 at digraph dot polyomino dot org dot uk> <6016E66D-2F43-4940-8289-C7AFA90112C6 at linux dot vnet dot ibm dot com> <alpine dot DEB dot 2 dot 20 dot 1606272121580 dot 3978 at digraph dot polyomino dot org dot uk>
On Mon, Jun 27, 2016 at 09:22:54PM +0000, Joseph Myers wrote:
> > > Ping. This patch
> > > <https://gcc.gnu.org/ml/gcc-patches/2016-06/msg01685.html> is pending
> > > review. Built-in functions are available in the followup patch
> > > <https://gcc.gnu.org/ml/gcc-patches/2016-06/msg01702.html>.
> >
> > I can't ack the patch, but the rs6000 bits of your original patch look fine.
> > I didn't mean to ask you to change those -- I hadn't read the patch
> > and was just commenting on your description of the patch, and I realize
> > now that I wasn't careful in my use of language. I apologize for the
> > miscommunication there. We should keep the check on
> > FLOAT128_IEEE_P to determine which mode is __float128 for now,
> > until the whole issue of two 128-bit floats is behind us.
>
> In that case, consider the relevant part of patch version 1 to be
> substituted into version 3 for the rest of the review.
The rs6000 part is approved, then.
Thanks,
Segher