[llvm] [SLP][REVEC] Remove assert because canConvertToMinOrMaxIntrinsic can support vector type. (PR #114946)

Alexey Bataev via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 5 08:18:16 PST 2024


================
@@ -10777,7 +10777,6 @@ BoUpSLP::getEntryCost(const TreeEntry *E, ArrayRef<Value *> VectorizedVals,
     // If the selects are the only uses of the compares, they will be
     // dead and we can adjust the cost by removing their cost.
     if (VI && SelectOnly) {
-      assert(!Ty->isVectorTy() && "Expected only for scalar type.");
----------------
alexey-bataev wrote:

Then you need to adjust the cost estimation here by changing the condition type. Also, this assertion guarantees, that it is not called for vector cost

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


More information about the llvm-commits mailing list