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

[Eugene Zelenko via Phabricator via cfe-commits]

Eugene.Zelenko added inline comments.


================
Comment at: clang-tools-extra/clang-doc/tool/ClangDocMain.cpp:205
+                       llvm::cl::OptionCategory &Category) {
+  auto OptionsParser =
+      CommonOptionsParser::create(argc, argv, Category, llvm::cl::ZeroOrMore);
----------------
Please don't use auto when type is not spelled in same statement.


================
Comment at: clang-tools-extra/clang-doc/tool/ClangDocMain.cpp:209
+    return OptionsParser.takeError();
+  for (auto I = ToolExecutorPluginRegistry::begin(),
+            E = ToolExecutorPluginRegistry::end();
----------------
Please use range loop.


https://reviews.llvm.org/D53170