sammccall accepted this revision.
sammccall added inline comments.
This revision is now accepted and ready to land.


================
Comment at: clang/tools/clang-check/ClangCheck.cpp:141
 
+class BuildSyntaxTree : public clang::ASTFrontendAction {
+public:
----------------
DumpSyntaxTree since this is used for its side-effect?


================
Comment at: clang/tools/clang-check/ClangCheck.cpp:152
+        clang::syntax::Arena A(AST.getSourceManager(), AST.getLangOpts(), TB);
+        llvm::outs() << clang::syntax::buildSyntaxTree(A, AST)->dump(
+            AST.getSourceManager());
----------------
ooh, we should consider colorizing this one day :-)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D95526/new/

https://reviews.llvm.org/D95526

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to