aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land.
LGTM with one minor testing request (conditional on the test passing, of course). ================ Comment at: unittests/Tooling/RecursiveASTVisitorTests/LambdaExpr.cpp:68 } TEST(RecursiveASTVisitor, VisitsAttributedLambdaExpr) { ---------------- Can you add a test that has a lambda within a lambda to ensure that everything is traversed as expected? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56444/new/ https://reviews.llvm.org/D56444 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits