[llvm] [VPlan] Compute cost of more replicating loads/stores in ::computeCost. (PR #160053)

[David Sherwood via llvm-commits]

================
@@ -1778,8 +1778,10 @@ VPCostContext::getOperandInfo(VPValue *V) const {
   return TTI::getOperandInfo(V->getLiveInIRValue());
 }
 
-InstructionCost VPCostContext::getScalarizationOverhead(
-    Type *ResultTy, ArrayRef<const VPValue *> Operands, ElementCount VF) {
+InstructionCost
+VPCostContext::getScalarizationOverhead(Type *ResultTy,
+                                        ArrayRef<const VPValue *> Operands,
+                                        ElementCount VF, bool Skip) {
----------------
david-arm wrote:

Should the variable be more descriptive, e.g. something like `SkipNonReplicatingLoadOpCost`? The reason I say this is because even when `Skip` is true we don't actually skip if it's a VPReplicateRecipe with opcode Load.

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