[PATCH] D78997: [SLP] add another bailout for load-combine patterns

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:3711-3722
+bool BoUpSLP::isLoadCombineCandidate() const {
+  // Peek through a final sequence of stores and check if all operations are
+  // likely to be load-combined.
+  unsigned NumElts = VectorizableTree[0]->Scalars.size();
+  for (Value *Scalar : VectorizableTree[0]->Scalars) {
+    Value *X;
+    if (!match(Scalar, m_Store(m_Value(X), m_Value())) ||
----------------
Maybe make it a part of `isTreeTinyAndNotFullyVectorizable()` function?


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

https://reviews.llvm.org/D78997