[clangd] Enable parsing of non-doxygen comments in global-symbol-builder

[ioeric]

Should we also change the behavior in dynamic index?

[ilya-biryukov]

In D47065#1104730, @ioeric wrote:

Should we also change the behavior in dynamic index?

Dynamic index already uses whatever AST provides (which is ParseAllComments = true since D46002)

[ioeric]

lgtm

clang-tools-extra/trunk/clangd/global-symbol-builder/GlobalSymbolBuilderMain.cpp

public:
             PragmaHandler(collectIWYUHeaderMaps(this->Includes.get())) {}
 
       std::unique_ptr<ASTConsumer>
       CreateASTConsumer(CompilerInstance &CI, StringRef InFile) override {
         CI.getPreprocessor().addCommentHandler(PragmaHandler.get());
         return WrapperFrontendAction::CreateASTConsumer(CI, InFile);
       }
 
+      bool BeginInvocation(CompilerInstance &CI) override {
+        // We want all comments, not just the doxygen ones.
+        CI.getLangOpts().CommentOpts.ParseAllComments = true;
+        return WrapperFrontendAction::BeginInvocation(CI);
+      }
+
       void EndSourceFileAction() override {
         WrapperFrontendAction::EndSourceFileAction();
 
         auto Symbols = Collector->takeSymbols();
         for (const auto &Sym : Symbols) {
           Ctx->reportResult(Sym.ID.str(), SymbolToYAML(Sym));
         }
       }