[llvm] 0dbcbfe - [VPlan] Don't assign slots for external defs (NFCI).

Florian Hahn via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 9 13:02:35 PDT 2023


Author: Florian Hahn
Date: 2023-04-09T21:01:21+01:00
New Revision: 0dbcbfe0d014bae2e00232366bfbae2914fd713e

URL: https://github.com/llvm/llvm-project/commit/0dbcbfe0d014bae2e00232366bfbae2914fd713e
DIFF: https://github.com/llvm/llvm-project/commit/0dbcbfe0d014bae2e00232366bfbae2914fd713e.diff

LOG: [VPlan] Don't assign slots for external defs (NFCI).

External defs are VPValues wrapping an IR value and hence will get
printed as ir<>. We don't need to assign a slot for a VPValue number.

Added: 
    

Modified: 
    llvm/lib/Transforms/Vectorize/VPlan.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/Vectorize/VPlan.cpp b/llvm/lib/Transforms/Vectorize/VPlan.cpp
index 84f577f0012c5..4655399cca73f 100644
--- a/llvm/lib/Transforms/Vectorize/VPlan.cpp
+++ b/llvm/lib/Transforms/Vectorize/VPlan.cpp
@@ -1108,10 +1108,6 @@ void VPSlotTracker::assignSlot(const VPValue *V) {
 }
 
 void VPSlotTracker::assignSlots(const VPlan &Plan) {
-
-  for (const auto &P : Plan.VPExternalDefs)
-    assignSlot(P.second);
-
   assignSlot(&Plan.VectorTripCount);
   if (Plan.BackedgeTakenCount)
     assignSlot(Plan.BackedgeTakenCount);


        


More information about the llvm-commits mailing list