[PATCH] D53170: [clang-doc] Switch to default to all-TUs executor

[Eric Liu via Phabricator via cfe-commits]

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