[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