[PATCH] D116740: [SLP]Improve reordering for the nodes beeing used in alternate vectorization.

Vasileios Porpodas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 6 10:23:51 PST 2022


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

LGTM



================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:3053
+      // Do not include ordering for nodes used in the alt opcode vectorization,
+      // better to reorder them during bottom-to-top stage.
+      unsigned Cnt = 0;
----------------
Please explain in the comment why this is better.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116740



More information about the llvm-commits mailing list