[PATCH] ASTMatcher for Attr

[Aaron Ballman]

On Thu, Aug 21, 2014 at 6:45 PM, Manuel Klimek <klimek at google.com> wrote:
>
>
> On Thu Aug 21 2014 at 3:37:22 PM Aaron Ballman <aaron at aaronballman.com>
> wrote:
>>
>> On Thu, Aug 21, 2014 at 5:56 PM, Manuel Klimek <klimek at google.com> wrote:
>> > +richard, since this is now touching lib/Basic...
>> >
>> > Looks good to me, but I want Richard to take a look whether this makes
>> > sense or should be solved differently.
>>
>> I am confused as to why the solution switched away from using the
>> AttrKind enum value and is now using a string.
>
>
> Note that the string is only for the dynamic matchers - the normal C++
> matchers still use the enum.

I hadn't noticed that; thank you for pointing it out! But at the risk
of demonstrating my ignorance, I guess I still don't understand why
the string is an improvement. AttrKinds.h is in Basic, so it's
available everywhere.

~Aaron