https://github.com/localspook created https://github.com/llvm/llvm-project/pull/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. >From 07c4f5b4d40be3c5e4285c2487aa6381bc278b7b Mon Sep 17 00:00:00 2001 From: Victor Chernyakin <[email protected]> Date: Fri, 12 Dec 2025 17:42:53 -0800 Subject: [PATCH] [clang-tidy][NFC] Remove obsolete FIXME comment --- clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.h | 4 ---- 1 file changed, 4 deletions(-) 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
