[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
================
@@ -2466,7 +2496,9 @@ class BoUpSLP {
}
// TODO: Check if we can remove a check for non-power-2 number of
// scalars after full support of non-power-2 vectorization.
- return UniqueValues.size() != 2 && has_single_bit(UniqueValues.size());
+ return UniqueValues.size() != 2 &&
+ hasFullVectorsOnly(*R.TTI, (*UniqueValues.begin())->getType(),
----------------
preames wrote:
Instead of white listing only the new class here, can we remove the restriction in a separate patch? This should be fully VL independent right?
https://github.com/llvm/llvm-project/pull/107273
More information about the llvm-commits
mailing list