[PATCH] D75286: [clangd] Handle clang-tidy suppression comments for diagnostics inside macro expansions
Sam McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat Mar 28 10:44:49 PDT 2020
sammccall added inline comments.
================
Comment at: clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.cpp:708-710
+ return std::tie(M1.FilePath, M1.FileOffset, LHS.DiagnosticName,
+ M1.Message) <
+ std::tie(M2.FilePath, M2.FileOffset, RHS.DiagnosticName, M2.Message);
----------------
nridge wrote:
> njames93 wrote:
> > This looks like a clang-format artifact, there are several other below. Could these be removed from this patch
> If you insist, I can move them to a separate patch. I don't want to leave it unmodified because the change will come back every time someone touches the file.
I don't personally care about this too much, but the changes are somewhat distracting in review, blame etc.
The policy we've mostly followed is to format changed lines only (git clang-format, turn format-on-save off) and leave misformatted code alone until it's next touched.
(Not sure if LLVM offers any guidance either way, but that's what Google does internally and IME it's been great)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75286/new/
https://reviews.llvm.org/D75286
More information about the cfe-commits
mailing list