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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
