[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 6 12:37:22 PDT 2024


================
@@ -5585,7 +5619,7 @@ void BoUpSLP::reorderTopToBottom() {
 
   // Reorder the graph nodes according to their vectorization factor.
   for (unsigned VF = VectorizableTree.front()->getVectorFactor(); VF > 1;
-       VF = bit_ceil(VF) / 2) {
+       VF = (VF & ~1U) - 2) {
----------------
preames wrote:

What is this trying to do?  & -1 should just be VF shouldn't it?

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


More information about the llvm-commits mailing list