[llvm] [JumpThreading] Invalidate LVI after `combineMetadataForCSE`. (PR #65219)

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 3 03:28:32 PDT 2023


================
@@ -115,6 +115,9 @@ class LazyValueInfo {
   /// PredBB to OldSucc to be from PredBB to NewSucc instead.
   void threadEdge(BasicBlock *PredBB, BasicBlock *OldSucc, BasicBlock *NewSucc);
 
+  /// Inform the analysis cache that we have erased a value.
----------------
nikic wrote:

For this usage we haven't actually erased a value, so this comment isn't quite right. Maybe "Remove information related to this value from the cache."

Could also call this API forgetValue() rather than eraseValue() to make it more generic.

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


More information about the llvm-commits mailing list