[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