[llvm] [VPlan] Speed up VPSlotTracker by using ModuleSlotTracker (PR #139881)

Igor Kirillov via llvm-commits llvm-commits at lists.llvm.org
Mon May 19 03:15:05 PDT 2025


================
@@ -1441,7 +1441,23 @@ void VPSlotTracker::assignName(const VPValue *V) {
   std::string Name;
   if (UV) {
     raw_string_ostream S(Name);
-    UV->printAsOperand(S, false);
+    if (MST) {
+      UV->printAsOperand(S, false, *MST);
----------------
igogo-x86 wrote:

@david-arm I’m not sure that actually helps much—MST is only used here and isn’t reused, so keeping the logic in one place feels simpler to me.

@fhahn, could you clarify what exactly you mean by “pulling out the code that gets the name for a VPValue”? I’m not sure which part you’re referring to or what the intended simplification would be.

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


More information about the llvm-commits mailing list