[clang] [clangl[TableGen] Change Diagnostic Emitter to use const RecordKeeper (PR #108209)

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 11 07:55:19 PDT 2024


================
@@ -255,20 +257,18 @@ class InferPedantic {
       GMap;
 
   DiagGroupParentMap &DiagGroupParents;
-  const std::vector<Record*> &Diags;
-  const std::vector<Record*> DiagGroups;
+  ArrayRef<const Record *> Diags;
----------------
AaronBallman wrote:

Are we sure this isn't going to cause lifetime issues because `ArrayRef` does not own its memory?

https://github.com/llvm/llvm-project/pull/108209


More information about the cfe-commits mailing list