[clang] [diagtool] Make the BuiltinDiagnosticsByID table sorted (PR #120321)
Karl-Johan Karlsson via cfe-commits
cfe-commits at lists.llvm.org
Wed Jan 15 22:27:27 PST 2025
================
@@ -23,28 +23,29 @@ llvm::ArrayRef<DiagnosticRecord> diagtool::getBuiltinDiagnosticsByName() {
return llvm::ArrayRef(BuiltinDiagnosticsByName);
}
-
// FIXME: Is it worth having two tables, especially when this one can get
// out of sync easily?
+// clang-format off
----------------
karka228 wrote:
"since this order needs to be preserved in multiple places, can you put together a wrapper header, which includes individual diagnostic kinds in the specific order and explain the relationship between these pieces?"
@kadircet I added a new file DiagnosticsID.inc as a "wrapper header".
https://github.com/llvm/llvm-project/pull/120321
More information about the cfe-commits
mailing list