[llvm] [SLPVectorizer] Use accurate cost for external users of resize shuffles (PR #137419)

Jeffrey Byrnes via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 28 18:08:17 PDT 2025


================
@@ -14802,7 +14803,39 @@ InstructionCost BoUpSLP::getTreeCost(ArrayRef<Value *> VectorizedVals,
                  << " for final shuffle of insertelement external users.\n";
           TE->dump(); dbgs() << "SLP: Current total cost = " << Cost << "\n");
       Cost += C;
-      return std::make_pair(TE, true);
+
----------------
jrbyrnes wrote:

Thanks -- I'll try to adopt it, but it's causing some concerning looking diffs in the lit tests that I need to work through.

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


More information about the llvm-commits mailing list