[llvm] [SLP] Normalize debug messages for newTreeEntry. (PR #119514)

Han-Kuan Chen via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 14 09:58:24 PST 2025


================
@@ -8417,13 +8418,14 @@ void BoUpSLP::buildTree_rec(ArrayRef<Value *> VL, unsigned Depth,
       }
       // Insert new order with initial value 0, if it does not exist,
       // otherwise return the iterator to the existing one.
-      newTreeEntry(VL, Bundle /*vectorized*/, S, UserTreeIdx,
-                   ReuseShuffleIndices, CurrentOrder);
+      TreeEntry *TE = newTreeEntry(VL, Bundle /*vectorized*/, S, UserTreeIdx,
+                                   ReuseShuffleIndices, CurrentOrder);
+      LLVM_DEBUG(dbgs() << "SLP: added a new TreeEntry "
+                           "(ExtractValueInst/ExtractElementInst).\n";
+                 TE->dump());
       // This is a special case, as it does not gather, but at the same time
       // we are not extending buildTree_rec() towards the operands.
-      ValueList Op0;
-      Op0.assign(VL.size(), VL0->getOperand(0));
-      VectorizableTree.back()->setOperand(0, Op0);
+      TE->setOperand(*this);
----------------
HanKuanChen wrote:

I use the commit 539b15b41a6a01017c0a555e89b7d2b62ba194d2 as base and revert 11676da80874787a47856da87911234837c53f06. However, `reduced.ll` gives me same result.

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


More information about the llvm-commits mailing list