eduucaldas added inline comments.
================
Comment at: clang/include/clang/AST/RecursiveASTVisitor.h:649
+ case BO_##NAME:
\
+ if (isSameMethod(&RecursiveASTVisitor::TraverseBin##NAME,
\
+ &Derived::TraverseBin##NAME)) {
\
----------------
There's only one place that call PostVisitStmt. Could we pull this conditional
there? It is repeated in every macro. It would make this function much simpler,
and make clear at call site what is being done
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82787/new/
https://reviews.llvm.org/D82787
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits