[llvm] [SLP] Fix cost estimation of external uses with wrong VF (PR #148185)

Mikael Holmén via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 15 00:02:32 PDT 2025


================
@@ -14632,6 +14630,11 @@ InstructionCost BoUpSLP::getTreeCost(ArrayRef<Value *> VectorizedVals,
   }
   SmallDenseSet<std::pair<Value *, Value *>, 8> CheckedScalarUser;
   for (ExternalUser &EU : ExternalUses) {
+    LLVM_DEBUG(dbgs() << "SLP: Computing cost for external use of TreeEntry "
+                      << EU.E.Idx << " in lane " << EU.Lane << "\n");
+    LLVM_DEBUG(dbgs() << "  User:" << *EU.User << "\n");
----------------
mikaelholmen wrote:

And now I realize that it's been fixed already in 4784ce9ebcf. Thanks @alexey-bataev 

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


More information about the llvm-commits mailing list