hans added a comment. I'd leave out the blank lines between DEF_TRAVERSE_STMTs, but the rest looks good.
================ Comment at: include/clang/AST/RecursiveASTVisitor.h:2014 @@ -2002,2 +2013,3 @@ DEF_TRAVERSE_STMT(ObjCAutoreleasePoolStmt, {}) + DEF_TRAVERSE_STMT(CXXForRangeStmt, { ---------------- I'm not sure about the blank lines in these DEF_TRAVERSE_STMTs. I don't really see what the logic is. Repository: rL LLVM http://reviews.llvm.org/D20254 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits