[llvm] [LV] Add scalar load/stores to VPReplicateRecipe::computeCost (PR #153218)

David Sherwood via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 4 05:01:19 PDT 2025


================
@@ -5205,6 +5205,8 @@ LoopVectorizationCostModel::getConsecutiveMemOpCost(Instruction *I,
     Cost += TTI.getMaskedMemoryOpCost(I->getOpcode(), VectorTy, Alignment, AS,
                                       CostKind);
   } else {
+    // TODO: Is this right? Operand 0 has a different meaning for loads and
+    // stores.
----------------
david-arm wrote:

Yeah, sorry just got caught up with other things. I guess I wasn't really sure whether you wanted me to change the documentation in this PR or were just pointing it out for information purpose?

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


More information about the llvm-commits mailing list