[PATCH] D100501: [cland] Dont emit missing newline warnings when building preamble
Kadir Cetinkaya via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Apr 23 09:58:08 PDT 2021
kadircet added a comment.
> Hit send too soon. But IIRC PP knows whether it's building a preamble.
That's a good point. But PP doesn't know if there's trailing content or not (in clangd's case).
So we'll end up not showing the diag even when we should (i.e. whole file is preamble without a new line at eof).
It doesn't sound too bad, but it'll effect all tools building preambles (including clang) and not just clangd so I am a little hesitant.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D100501/new/
https://reviews.llvm.org/D100501
More information about the cfe-commits
mailing list