From 0019684900491f517f3b08b4fa92740b69a8cc0f Mon Sep 17 00:00:00 2001 From: Ilya Biryukov Date: Thu, 7 Nov 2019 09:53:07 +0100 Subject: [PATCH] [clangd] Set RetainCommentsFromSystemHeaders to true clangd should retain comments from system headers. fixes https://github.com/clangd/clangd/issues/96 Patch by lh123! Differential revision: https://reviews.llvm.org/D69928 --- clang-tools-extra/clangd/Compiler.cpp | 1 + clang-tools-extra/clangd/index/IndexAction.cpp | 1 + 2 files changed, 2 insertions(+) diff --git a/clang-tools-extra/clangd/Compiler.cpp b/clang-tools-extra/clangd/Compiler.cpp index e080143..795fd0082 100644 --- a/clang-tools-extra/clangd/Compiler.cpp +++ b/clang-tools-extra/clangd/Compiler.cpp @@ -63,6 +63,7 @@ buildCompilerInvocation(const ParseInputs &Inputs, // createInvocationFromCommandLine sets DisableFree. CI->getFrontendOpts().DisableFree = false; CI->getLangOpts()->CommentOpts.ParseAllComments = true; + CI->getLangOpts()->RetainCommentsFromSystemHeaders = true; return CI; } diff --git a/clang-tools-extra/clangd/index/IndexAction.cpp b/clang-tools-extra/clangd/index/IndexAction.cpp index 0814e7f..8fd2159 100644 --- a/clang-tools-extra/clangd/index/IndexAction.cpp +++ b/clang-tools-extra/clangd/index/IndexAction.cpp @@ -160,6 +160,7 @@ public: bool BeginInvocation(CompilerInstance &CI) override { // We want all comments, not just the doxygen ones. CI.getLangOpts().CommentOpts.ParseAllComments = true; + CI.getLangOpts().RetainCommentsFromSystemHeaders = true; // Index the whole file even if there are warnings and -Werror is set. // Avoids some analyses too. Set in two places as we're late to the party. CI.getDiagnosticOpts().IgnoreWarnings = true; -- 2.7.4