[cfe-commits] r63519 - in /cfe/trunk: lib/Sema/SemaDecl.cpp test/Sema/static-init.c

Eli Friedman eli.friedman at gmail.com
Mon Feb 2 13:19:27 PST 2009


On Mon, Feb 2, 2009 at 1:02 PM, Douglas Gregor <dgregor at apple.com> wrote:
>
> On Feb 2, 2009, at 12:55 PM, Eli Friedman wrote:
>
>> On Mon, Feb 2, 2009 at 12:20 PM, Douglas Gregor <dgregor at apple.com> wrote:
>>>
>>> On Feb 2, 2009, at 8:07 AM, Nuno Lopes wrote:
>>>
>>>> Author: nlopes
>>>> Date: Mon Feb  2 10:07:41 2009
>>>> New Revision: 63519
>>>>
>>>> URL: http://llvm.org/viewvc/llvm-project?rev=63519&view=rev
>>>> Log:
>>>> allow cast from array to int to be considered as constant
>>>
>>> Is this cast permitted by the standard, or is it a GNU extension?
>>
>> Extension, but as far as I know, it's almost universally supported.
>
>
> A comment or warning here would really help. We have an opportunity to
> conform as best we can to the standard, and thereby help others conform
> better (or fix the standard), and we should take that opportunity, IMO.

The constant-initializer checking code in particular is a mess at the
moment; fixing it is on my todo list one PR3398 is fixed.

-Eli



More information about the cfe-commits mailing list