[llvm] [VPlan] Version VPValue names in VPSlotTracker. (PR #81411)

via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 12 15:14:29 PDT 2024


================
@@ -1373,32 +1362,84 @@ VPInterleavedAccessInfo::VPInterleavedAccessInfo(VPlan &Plan,
   visitRegion(Plan.getVectorLoopRegion(), Old2New, IAI);
 }
 
-void VPSlotTracker::assignSlot(const VPValue *V) {
-  if (V->getUnderlyingValue())
+void VPSlotTracker::assignSlotOrName(const VPValue *V) {
+  auto *UV = V->getUnderlyingValue();
+  if (!UV) {
+    assert(!VPValue2Name.contains(V) && "VPValue already has a slot!");
+    VPValue2Name[V] = (Twine("vp<%") + Twine(NextSlot) + ">").str();
+    NextSlot++;
     return;
-  assert(!Slots.contains(V) && "VPValue already has a slot!");
-  Slots[V] = NextSlot++;
+  }
----------------
ayalz wrote:

Worth adding a comment, e.g.,
// Use the name of the underlying Value, wrapped in "ir<>", and versioned.

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


More information about the llvm-commits mailing list