================
@@ -401,6 +410,7 @@ class FactManager {
   /// Facts for each CFG block, indexed by block ID.
   llvm::SmallVector<llvm::SmallVector<const Fact *>> BlockToFacts;
   llvm::BumpPtrAllocator FactAllocator;
+  std::optional<llvm::BitVector> PersistentOrigins;
----------------
usx95 wrote:

Since `LoanPropagation` and `LiveOrigins` are not optional analyses, I think we 
can
- remove the optional here. 
- construct this unconditionally in 
`FactManager::computePersistentOrigins(const CFG&)`
- make `FactManager::getPersistentOrigins()` simply a getter instead of current 
getOrCreate semantics.

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

Reply via email to