[llvm] [SLP]Initial support for non-power-of-2 (but still whole register) number of elements in operands. (PR #107273)

Alexey Bataev via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 16 12:12:37 PDT 2024


================
@@ -5431,7 +5465,7 @@ BoUpSLP::getReorderingData(const TreeEntry &TE, bool TopToBottom) {
 
     // FIXME: Remove the non-power-of-two check once findReusedOrderedScalars
     // has been auditted for correctness with non-power-of-two vectors.
-    if (!TE.isNonPowOf2Vec())
+    if (!TE.hasNonWholeRegisterElems(*TTI))
----------------
alexey-bataev wrote:

Reordering requires some extra testing, reverting it for now

https://github.com/llvm/llvm-project/pull/107273


More information about the llvm-commits mailing list