[PATCH] D24268: Traversing template paramter lists of DeclaratorDecls and/or TagDecls.
Richard Smith via cfe-commits
cfe-commits at lists.llvm.org
Tue Sep 6 18:18:40 PDT 2016
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
This patch looks great, thank you! Do you have an SVN account or do you need someone to commit this for you?
================
Comment at: unittests/ASTMatchers/ASTMatchersTraversalTest.cpp:729
@@ -645,2 +728,2 @@
EXPECT_TRUE(matches("template<int T> struct C {}; C<42> c;",
classTemplateSpecializationDecl(
----------------
I'd view the `hasAncestor` failure as an indirect symptom of the root problem, which is that we were failing to traverse some nodes. I prefer the new `hasName` test, since it more directly checks for that immediate problem.
https://reviews.llvm.org/D24268
More information about the cfe-commits
mailing list