[PATCH] D125111: [SLP] Make reordering aware of external vectorizable scalar stores.

Valeriy Dmitriev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 10 07:59:19 PDT 2022


vdmitrie accepted this revision.
vdmitrie added a comment.
This revision is now accepted and ready to land.

Looks good with a nit.



================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:4119
+
+LLVM_DUMP_METHOD static void dumpOrder(const BoUpSLP::OrdersType &Order) {
+  for (unsigned Idx : Order)
----------------
wrap it with  #ifndef NDEBUG (or delete).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D125111/new/

https://reviews.llvm.org/D125111



More information about the llvm-commits mailing list