[cfe-dev] Adding access to array size expressions in ConstantArrayType.
Enea Zaffanella
zaffanella at cs.unipr.it
Sat Jul 4 00:31:12 PDT 2009
Enea Zaffanella wrote:
> Eli Friedman wrote:
>> On Tue, Jun 30, 2009 at 12:35 PM, Enea
>> Zaffanella<zaffanella at cs.unipr.it> wrote:
>
> [...]
>
>> Unlike the other array types you're adding Brackets to,
>> IncompleteArrayType gets canonicalized; we don't want a source range
>> on a canonical IncompleteArrayType, and we don't want to end up with
>> the wrong SourceRange on a non-canonical IncompleteArrayType. It's a
>> little tricky, so if the issues I'm talking about aren't obvious, I'd
>> suggest skipping adding the bracket locations to IncompleteArrayType
>> for now.
>>
>> Otherwise, this is looking good!
>>
>> -Eli
>
> OK, here is the patch revised along your suggestions.
>
> Enea.
I got no feedback on my last revision of the patch and it seems that the
patch was not applied. Is there something wrong with it? Or should I
just wait a bit more?
Cheers,
Enea.
More information about the cfe-dev
mailing list