arphaman accepted this revision. arphaman added a comment. This revision is now accepted and ready to land.
Lgtm with two requests below ================ Comment at: include/clang/AST/RecursiveASTVisitor.h:354 +protected: + bool TraverseStmtBase(Stmt *S, DataRecursionQueue *Queue) { + return TRAVERSE_STMT_BASE(Stmt, Stmt, S, Queue); ---------------- I don't think you need this function now ================ Comment at: include/clang/AST/RecursiveASTVisitor.h:2089 } - DEF_TRAVERSE_STMT(GCCAsmStmt, { ---------------- Please avoid deleting this line https://reviews.llvm.org/D37200 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits