[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