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

[Julie Hockett via Phabricator via cfe-commits]

juliehockett marked 3 inline comments as done.
juliehockett added inline comments.


================
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) {
----------------
ioeric wrote:
> 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()`.
Perfect. Thanks!


https://reviews.llvm.org/D53170