alexfh added inline comments.
================ Comment at: clang-tidy/modernize/UseAutoCheck.cpp:173-177 +/// Matches the type that was substituted for the template parameter. +AST_MATCHER_P(SubstTemplateTypeParmType, hasReplacementType, + ast_matchers::internal::Matcher<QualType>, InnerMatcher) { + return InnerMatcher.matches(Node.getReplacementType(), Finder, Builder); +} ---------------- Ideally, this should go to ASTMatchers.h (with a proper test and documentation). ================ Comment at: clang-tidy/modernize/UseAutoCheck.cpp:256 +StatementMatcher makeDeclWithTemplateCastMatcher() { + auto ST = substTemplateTypeParmType(hasReplacementType(equalsBoundNode("a"))); + ---------------- It might make sense to use a more specific identifier than "a" to avoid collisions. https://reviews.llvm.org/D27166 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits