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

Reply via email to