This is the mail archive of the newlib@sourceware.org mailing list for the newlib project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

RE: long double (was "strtold?")


Umm, I don't understand the change to machine/ieeefp.h.  The defines
that it was using are the official names from float.h.  Adding __ as a
prefix and suffix ties it directly to gcc.  The makes it work for gcc
without float.h having been included, but won't it break other
compilers?
(There's nothing in the standards that I know of that would require
__LDBL_MANT_DIG__ to be defined, for example.)  So perhaps it did not
build as it was, but the way to fix it for all compilers is to add
#include <float.h>, I think.  (Newlib doesn't require gcc, does it?)
				Craig

-----Original Message-----
From: newlib-owner@sourceware.org [mailto:newlib-owner@sourceware.org]
On Behalf Of Jeff Johnston
Sent: Friday, April 17, 2009 4:55 PM
To: Ken Werner
Cc: newlib@sourceware.org
Subject: Re: long double (was "strtold?")

Ken Werner wrote:
> On Thursday 16 April 2009 20:25:17 you wrote:
>   
>> Checked in with minor tweaks to ChangeLog and libm.texinfo.
>>
>> -- Jeff J.
>>     
>
> Hi,
>
> I noticed two glitches with the current long double wrappers. First,
the check 
> whether long double is as wide as double within ieeefp.h uses wrong
defines - I 
> missed some underscores. Second, the llrintl prototype and the
corresponding 
> wrapper function was missing. The attached patch fixes these two
issues. Sorry 
> for the inconvenience caused.
>
> Ken
>
>   
Patch checked in.

-- Jeff J.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]