================
@@ -167,7 +167,7 @@ class IndexAction : public ASTFrontendAction {
   bool BeginInvocation(CompilerInstance &CI) override {
     // We want all comments, not just the doxygen ones.
     CI.getLangOpts().CommentOpts.ParseAllComments = true;
-    CI.getLangOpts().RetainCommentsFromSystemHeaders = true;
+    CI.getLangOpts().CommentOpts.RetainCommentsFromSystemHeaders = true;
----------------
AnonMiraj wrote:

since ParseAllComments is already true wouldn't this be redundant?
```cpp
bool Sema::shouldRetainCommentsInAST(SourceLocation Loc) const {
  if (LangOpts.CommentOpts.ParseAllComments) return true; 
  if (LangOpts.CommentOpts.RetainComments)   return true;
  ...
}
```

https://github.com/llvm/llvm-project/pull/206363
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to