alexfh added a comment.

Another couple of nits.

Comment at: docs/HowToSetupToolingForLLVM.rst:136
       if (this->ASTDump.operator _Bool())
-          return clang::CreateASTDumper(this->ASTDumpFilter);
+          return clang::CreateASTDumper(nullptr, this->ASTDumpFilter);
       if (this->ASTPrint.operator _Bool())
Please add an argument comment for nullptr, e.g.

  return clang::CreateASTDumper(/*OS=*/nullptr, ...);

or maybe

  return clang::CreateASTDumper(nullptr /*dump to stderr*/, ...);

Same in the actual callers below.

Comment at: include/clang/Frontend/ASTConsumers.h:38
+// AST dumper: dumps the raw AST in human-readable form to the given output
+// stream.
+std::unique_ptr<ASTConsumer> CreateASTDumper(std::unique_ptr<raw_ostream> OS,
Add "When OS is nullptr, dumps AST to stderr."

  rC Clang

cfe-commits mailing list

Reply via email to