[PATCH] Add loc() to the dynamic registry.

Manuel Klimek klimek at google.com
Mon Mar 10 06:59:09 PDT 2014


On Mon, Mar 10, 2014 at 2:55 PM, Samuel Benzaquen <sbenza at google.com> wrote:

>
>
> ================
> Comment at: include/clang/ASTMatchers/ASTMatchers.h:2829
> @@ -2827,1 +2828,3 @@
> +typedef internal::BindableMatcher<TypeLoc>(&loc_Type0)(
> +    const internal::Matcher<QualType> &);
>
> ----------------
> Manuel Klimek wrote:
> > I'd vote for putting those into ASTMatchersInternal.h and into the
> internal namespace.
> ... maybe this should be defined with a macro like many other matchers.
> That way you don't even see this quirk of the registration system.
> I'll try that and ping you again.
>

sg


>
>
> http://llvm-reviews.chandlerc.com/D2928
>
> BRANCH
>   loc
>
> ARCANIST PROJECT
>   clang
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140310/73835be5/attachment.html>


More information about the cfe-commits mailing list