[cfe-dev] Adding access to array size expressions in ConstantArrayType.
eli.friedman at gmail.com
Sat Jul 4 03:31:47 PDT 2009
On Sat, Jul 4, 2009 at 12:31 AM, Enea Zaffanella<zaffanella at cs.unipr.it> wrote:
> Enea Zaffanella wrote:
>> Eli Friedman wrote:
>>> On Tue, Jun 30, 2009 at 12:35 PM, Enea Zaffanella<zaffanella at cs.unipr.it>
>>> 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!
>> OK, here is the patch revised along your suggestions.
> 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?
Nothing wrong with it; I'll take a final look at it and apply it
sometime in the next couple days.
More information about the cfe-dev