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

Alexander Kornienko via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 14 10:32:59 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);
----------------
alexfh wrote:

I confirmed this at b270525f730be6e7196667925f5a9bfa153262e9, but at 283dca56f8dddbf2f144730a01675c94b04f57cb this already seems to have been fixed. Searching for the fix.

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


More information about the llvm-commits mailing list