[cfe-commits] r61444 - /cfe/trunk/lib/Headers/float.h

Eli Friedman eli.friedman at gmail.com
Fri Dec 26 15:00:20 PST 2008


On Fri, Dec 26, 2008 at 2:48 PM, Anders Carlsson <andersca at mac.com> wrote:
>
> On Friday, December 26, 2008, at 01:23PM, "Eli Friedman" <eli.friedman at gmail.com> wrote:
>>On Fri, Dec 26, 2008 at 9:10 AM, Anders Carlsson <andersca at mac.com> wrote:
>>> Author: andersca
>>> Date: Fri Dec 26 11:10:18 2008
>>> New Revision: 61444
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=61444&view=rev
>>> Log:
>>> Add float.h header. Eli, please review :)
>>
>>There's not much to say... I think we were planning at one point to do
>>this a bit differently so that we didn't have to define so many
>>identifiers on initialization, but this works.
>>
>
> Oh, how?

Roughly, define a new function, __builtin_limit(), or something like
that, which the preprocessor expands, that allows queries for these
sorts of things.  That way, we can get rid of the mess of
DefineBuiltinMacro calls in Preprocessor.cpp.  I suppose it's not a
particularly high priority, though.

> (I think it's good to have at least something in place so we won't have to depend on GCC)

Agreed.

-Eli



More information about the cfe-commits mailing list