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

Reply via email to