[llvm] [LV] Use getFixedValue instead of getKnownMinValue when appropriate (PR #143526)

David Sherwood via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 10 07:34:59 PDT 2025


================
@@ -5342,6 +5342,10 @@ LoopVectorizationCostModel::getUniformMemOpCost(Instruction *I,
   StoreInst *SI = cast<StoreInst>(I);
 
   bool IsLoopInvariantStoreValue = Legal->isInvariant(SI->getValueOperand());
+  // TODO: We have tests that request the cost of extracting element
+  // VF.getKnownMinValue() - 1 from a scalable vector. This is actually
+  // meaningless, given what we actually want is the last lane and is likely
+  // to be more expensive.
----------------
david-arm wrote:

I don't think there is such a TTI hook. Maybe an argument for adding one, or at least modifying an existing interface? I'm happy to look into it.

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


More information about the llvm-commits mailing list