kadircet added inline comments.
================
Comment at: clang-tools-extra/clangd/Diagnostics.cpp:563
FillDiagBase(*LastDiag);
- adjustDiagFromHeader(*LastDiag, Info, *LangOpts);
+ if (!InsideMainFile)
+ LastDiagWasAdjusted = adjustDiagFromHeader(*LastDiag, Info, *LangOpts);
----------------
ilya-biryukov wrote:
> kadircet wrote:
> > ilya-biryukov wrote:
> > > We probably want to **always** set the value of this field to avoid
> > > accidentally reading the flag for the previous `LastDiag`
> > I don't follow, this is always set in line `475` at the beginning of
> > `HandleDiagnostic` method.
> I might be missing something, but the only assignment to
> `LastDiagWasAdjusted` seems to be here (apart from initialization at the
> declaration site)
oopsy, fixed
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64863/new/
https://reviews.llvm.org/D64863
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
