[llvm] [SLP]Initial support for non-power-of-2 (but still whole register) number of elements in operands. (PR #107273)
Philip Reames via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 13 15:23:39 PDT 2024
================
@@ -3390,7 +3423,8 @@ class BoUpSLP {
Last->State = EntryState;
// FIXME: Remove once support for ReuseShuffleIndices has been implemented
// for non-power-of-two vectors.
- assert((has_single_bit(VL.size()) || ReuseShuffleIndices.empty()) &&
+ assert((hasFullVectorsOnly(*TTI, getValueType(VL.front()), VL.size()) ||
----------------
preames wrote:
Have you audited the reordering code for VL=N*VLMAX? I suspect not as it's not trivial, and I'd expect it to require changes. If you have, we should be able to just remove this assert instead as it should work for any VL.
https://github.com/llvm/llvm-project/pull/107273
More information about the llvm-commits
mailing list