[VPlan] Strip stale comment for SCEVToExpansion (NFC)

[artagnon]

ExpandSCEV recipes are expanded late, and SCEVToExpansion is just a cache that is not scheduled for removal.

[llvmbot]

@llvm/pr-subscribers-llvm-transforms

@llvm/pr-subscribers-vectorizers

Author: Ramkumar Ramachandra (artagnon)

Changes

ExpandSCEV recipes are expanded late, and SCEVToExpansion is just a cache that is not scheduled for removal.

---

Full diff: https://github.com/llvm/llvm-project/pull/164288.diff

1 Files Affected:

• (modified) llvm/lib/Transforms/Vectorize/VPlan.h (-2)

diff --git a/llvm/lib/Transforms/Vectorize/VPlan.h b/llvm/lib/Transforms/Vectorize/VPlan.h
index fed04eb1cbe9f..2eeaee712690a 100644
--- a/llvm/lib/Transforms/Vectorize/VPlan.h
+++ b/llvm/lib/Transforms/Vectorize/VPlan.h
@@ -4164,8 +4164,6 @@ class VPlan {
   SmallVector<VPValue *, 16> VPLiveIns;
 
   /// Mapping from SCEVs to the VPValues representing their expansions.
-  /// NOTE: This mapping is temporary and will be removed once all users have
-  /// been modeled in VPlan directly.
   DenseMap<const SCEV *, VPValue *> SCEVToExpansion;
 
   /// Blocks allocated and owned by the VPlan. They will be deleted once the

[fhahn]

This is an interesting one, originally I was a bit surprised and while trying to confirm if the TODO is still needed it turns out the while map isn't needed any longer: https://github.com/llvm/llvm-project/pull/164490/files