================
@@ -97,6 +105,7 @@ class LifetimeSafetyAnalysis {
   FactManager &getFactManager() { return *FactMgr; }
 
 private:
+  const LifetimeSafetyOpts &LSOpts;
----------------
Xazax-hun wrote:

I think it might be safer if we actually stored a copy. I think with the 
current API it would be tempting to just pass in a temporary to the constructor 
and have a dangling reference. 

https://github.com/llvm/llvm-project/pull/170444
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to