[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