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

[Mikael Holmén via llvm-commits]

================
@@ -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:

Hi @gbossu

I've seen these new debug printouts crash here when "EU.User" is null.
Unfortunately I don't have a reproducer to share.

Should this never happen or should the debug printout be fixed?

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