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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
