[llvm] [ARM][SLP] Fix cost function for SLP Vectorization of ZExt/SExt (PR #122713)
David Green via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 20 03:21:54 PST 2025
https://github.com/davemgreen commented:
The MVE instruction that getExtendedReductionCost is trying to model is vaddlv, which is a `i64 vecreduce(v4i64 sext(v4i32))`. So it would seem odd to add the cost of the extend again to it. BaseT::getExtendedReductionCost includes the cast cost too, so it would look like this is double-counting.
This is an example of MVE using the vaddlv instruction, which looks decent in this small example: https://llvm.godbolt.org/z/oTrMTcM4E
https://github.com/llvm/llvm-project/pull/122713
More information about the llvm-commits
mailing list