[PATCH] D136467: [SLP] Correct shuffle cost to reuse vectorized values in depending nodes

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 21 10:08:23 PDT 2022


ABataev added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:6799-6811
+        TargetTransformInfo::ShuffleKind ShuffleKind =
+            TargetTransformInfo::SK_Select;
+        const TreeEntry *TE = nullptr;
+        for (unsigned I = 0, N = E->getNumOperands(); I < N; ++I) {
+          const TreeEntry *Op = getVectorizedOperand(E, I);
+          if (Op && !TE)
+            TE = Op;
----------------
Can you try to reuse buildShuffleEntryMask instead?


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

https://reviews.llvm.org/D136467



More information about the llvm-commits mailing list