johannes added inline comments.
================ Comment at: include/clang/AST/LexicallyOrderedRecursiveASTVisitor.h:75 + Derived &getDerived() { return *static_cast<Derived *>(this); } + ---------------- arphaman wrote: > I don't think you need this since `getDerived` in RecursiveASTVisitor is > already public. `has_same_member_pointer` and `getDerived` are used by the TRAVERSE_STMT macro, which should be the right one to use here. Also the `RecursiveASTVisitor` typedef just above. ================ Comment at: include/clang/AST/RecursiveASTVisitor.h:3212 #undef TRAVERSE_STMT -#undef TRAVERSE_STMT_BASE ---------------- arphaman wrote: > Getting rid of `undef` is not ideal. You might want to extract these macros > into one .def file that's included by both RecursiveASTVisitor.h and the > LexicallyOrdered one. Ok, I will do that https://reviews.llvm.org/D37200 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits