[llvm] [VPlan] Remove no longer needed VP intrinsic handling in VPWidenIntrinsicRecipe::computeCost. NFCI (PR #137573)
Luke Lau via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 29 06:47:26 PDT 2025
lukel97 wrote:
> LGTM. There is an unrelated question: is there a possibility that the operand is not of vector type?
>
> ```
> SmallVector<Type *> ParamTys;
> for (unsigned I = 0; I != getNumOperands(); ++I)
> ParamTys.push_back(
> toVectorTy(Ctx.Types.inferScalarType(getOperand(I)), VF));
> ```
Yes I think so. I'm not sure why this isn't an issue at the moment. But we should be checking `onlyFirstLaneUsed` here probably.
https://github.com/llvm/llvm-project/pull/137573
More information about the llvm-commits
mailing list