This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCHv4] ldbl-128: Use L(x) macro for long double constants
On Thu, 8 Sep 2016, Paul E. Murphy wrote:
> On 09/08/2016 05:56 PM, Joseph Myers wrote:
> > On Fri, 2 Sep 2016, Paul E. Murphy wrote:
> >
> >> @@ -65,7 +65,7 @@ __ieee754_atan2l(_Float128 y, _Float128 x)
> >> if(((ix|((lx|-lx)>>63))>0x7fff000000000000LL)||
> >> ((iy|((ly|-ly)>>63))>0x7fff000000000000LL)) /* x or y is NaN */
> >> return x+y;
> >> - if(((hx-0x3fff000000000000LL)|lx)==0) return __atanl(y); /* x=1.0L */
> >> + if(((hx-0x3fff000000000000LL)|lx)==0) return __atanl(y); /* x=1 */
> >
> > Is this change of a comment deliberate? I thought you were avoiding
> > changing comments.
>
> This is a limitation of the conversion script. It only skips lines
> which start with a /* comment. This can be reverted via the fixup,
> patch or I can note the limitation in the commit message.
The patch is OK with the comment change reverted.
--
Joseph S. Myers
joseph@codesourcery.com