ioeric requested changes to this revision.
ioeric added inline comments.
This revision now requires changes to proceed.
================
Comment at: clang-tools-extra/clang-doc/tool/ClangDocMain.cpp:203
+llvm::Expected<std::unique_ptr<ToolExecutor>>
+createClangDocExecutor(int &argc, const char **argv,
+ llvm::cl::OptionCategory &Category) {
----------------
This is exposing too much implementation details.I landed a patch to expose the
`ExecutorName` option from the library (rL344335) so you could simply do
`ExecutorName.setInitialValue("all-TUs")` before calling
`createExecutorFromCommandLineArgs()`.
https://reviews.llvm.org/D53170
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
