Author: Victor Chernyakin Date: 2025-12-13T16:28:09-08:00 New Revision: 49ad1e9ea25b8524b0ab1d61ca5fa16103434357
URL: https://github.com/llvm/llvm-project/commit/49ad1e9ea25b8524b0ab1d61ca5fa16103434357 DIFF: https://github.com/llvm/llvm-project/commit/49ad1e9ea25b8524b0ab1d61ca5fa16103434357.diff LOG: [clang-tidy][NFC] Remove obsolete FIXME comment (#172120) This comment was written 12 years ago. It's no longer correct to say that diagnostic reporting is under heavy development, and we seem to be doing just fine without tablegenned IDs, so I think we can simply remove it. Added: Modified: clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.h Removed: ################################################################################ diff --git a/clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.h b/clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.h index 21ffd9de35c19..8de5778dfefb0 100644 --- a/clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.h +++ b/clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.h @@ -91,10 +91,6 @@ class ClangTidyContext { ClangTidyContext &operator=(const ClangTidyContext &) = delete; /// Report any errors detected using this method. - /// - /// This is still under heavy development and will likely change towards using - /// tablegen'd diagnostic IDs. - /// FIXME: Figure out a way to manage ID spaces. DiagnosticBuilder diag(StringRef CheckName, SourceLocation Loc, StringRef Description, DiagnosticIDs::Level Level = DiagnosticIDs::Warning); _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
