aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land.
LGTM with one minor correction (you can correct when committing it). ================ Comment at: unittests/ASTMatchers/ASTMatchersTest.cpp:2343 @@ +2342,3 @@ + +TEST(DestructorDeclaration, MatchesVirtualDestructor) { + EXPECT_TRUE(matches("class Foo { virtual ~Foo(); };", ---------------- I think this test got duplicated accidentally; I don't see it removed as part of the diff. http://reviews.llvm.org/D18584 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits