[cfe-dev] AST matcher for C++11 [[attributes]]

[Aaron Ballman via cfe-dev]

On Thu, Feb 2, 2017 at 7:33 AM, Andrzej Krzemienski via cfe-dev
<cfe-dev at lists.llvm.org> wrote:
>
>
> 2017-01-23 15:14 GMT+01:00 Aaron Ballman <aaron at aaronballman.com>:
>>
>> On Mon, Jan 23, 2017 at 6:13 AM, Andrzej Krzemienski via cfe-dev
>> <cfe-dev at lists.llvm.org> wrote:
>> > Hi everyone,
>> > I am not sure of this is the right list. I have an issue with
>> > clang-tidy's
>> > predefined AST Matchers. The documentation for using clang-tidy
>> > (http://clang.llvm.org/extra/clang-tidy/) suggests that one should first
>> > try
>> > to use of predefined matchers listed in
>> > http://clang.llvm.org/docs/LibASTMatchersReference.html.
>> >
>> > But there is no matcher for C++11 [[attributes]]. There is matcher
>> > hasAttr,
>> > but it only works for predefined GCC __attribute__((s)).
>>
>> The hasAttr() matcher works for all attributes based on the Attr::Kind
>> enumeration. This will not distinguish between different spellings of
>> the same attribute, but that should (hopefully) be largely irrelevant.
>> So this matcher should do what you need already.
>
>
> Aaron, thanks for the reply. However it seams I phrased my question
> incorrectly. I would like to check the existence of my own unique attribute,
> say [[ab::cd::my_attribute]]. It is not in the list of any known attributes.
> hasAttr() appears to only detect the known attributes that are already
> listed in the long attribute enum. But will not see a new attribute.
>
> So my question is, in clang-tidy is it possible to inspect custom
> attributes, like [[ab::cd::my_attribute]] ?

If your custom attribute is implemented by adding the attribute to
Attr.td, then it should work fine (depending on your attribute
definition). The list of known attributes used by hasAttr() is
automatically generated from the attributes listed in Attr.td, and the
only attributes that don't get enumerated are ones which do not have
an AST node associated with them.

If your attribute is not added to Attr.td, then you are correct,
clang-tidy has no knowledge of it.

~Aaron

>
> Regards,
> &rzej;
>
>
> _______________________________________________
> cfe-dev mailing list
> cfe-dev at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev
>