[PATCH] D51880: [ASTMatchers] add two matchers for dependent expressions
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Sep 11 05:19:06 PDT 2018
aaron.ballman added a comment.
In https://reviews.llvm.org/D51880#1230221, @JonasToth wrote:
> In https://reviews.llvm.org/D51880#1229513, @aaron.ballman wrote:
>
> > Missing tests and changes to Registry.cpp for dynamic matchers.
> >
> > Also, do you want to add `isInstantiationDependent()` at the same time, given the relationship with the other two matchers?
>
>
> Do you mean a matcher that does `return Node.isValueDependent() || Node.isTypeDependent()` or `hasAncestor(expr(anyOf(isValueDependent(), isTypeDependent())))`?
I mean a matcher that does `return Node.isInstantiationDependent();` over `Expr`.
Repository:
rC Clang
https://reviews.llvm.org/D51880
More information about the cfe-commits
mailing list