[PATCH] D122148: [SLP] Peek into loads when hitting the RecursionMaxDepth

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 5 10:36:02 PDT 2022


ABataev added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:4119-4125
+         all_of(VL,
+                [](const Value *I) {
+                  return match(I,
+                               m_OneUse(m_ZExt(m_OneUse(m_Load(m_Value())))));
+                }) ||
+         all_of(VL, [](const Value *I) {
+           return match(I, m_OneUse(m_SExt(m_OneUse(m_Load(m_Value())))));
----------------
Can you merge these 2 checks into one with single `all_of` to avoid 2 similar checks?


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

https://reviews.llvm.org/D122148



More information about the llvm-commits mailing list