[PATCH] D51880: [ASTMatchers] add two matchers for dependent expressions

Jonas Toth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 11 03:31:06 PDT 2018


JonasToth added a comment.

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())))`?


Repository:
  rC Clang

https://reviews.llvm.org/D51880





More information about the cfe-commits mailing list