[PATCH] D19841: [clang-tidy] Lift common matchers to utils namespace

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Tue May 17 12:02:03 PDT 2016


aaron.ballman added inline comments.

================
Comment at: clang-tidy/utils/Matchers.h:20
@@ -19,1 +19,3 @@
 
+AST_MATCHER_P(Expr, ignoringImplicit,
+              ast_matchers::internal::Matcher<Expr>, InnerMatcher) {
----------------
alexfh wrote:
> Consider moving this to ASTMatchers.h in a follow-up.
Honestly, this one is a lot less interesting as a public AST matcher. I would avoid that unless there's a really solid case for it.


http://reviews.llvm.org/D19841





More information about the cfe-commits mailing list