[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