[llvm] [VPlan] Process simplifyRecipes via a worklist (PR #133977)

Florian Hahn via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 13 12:54:22 PDT 2025


================
@@ -63,6 +63,9 @@ class VPTypeAnalysis {
 
   /// Return the LLVMContext used by the analysis.
   LLVMContext &getContext() { return Ctx; }
+
+  /// Remove \p V from the cache. You must call this after a value is erased.
----------------
fhahn wrote:

```suggestion
  /// Remove \p V from the cache. You must call this after a VPValue in the cache is erased.
```

(I think just erasing isn't a problem, the problem is when new VPValues/recipes are created and added to the cache, if they get allocated to the same address as a previous cached VPValue. But that's too detailed ;))

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


More information about the llvm-commits mailing list