[PATCH] D140580: [SLP]Do not emit many extractelements, reuse the single one emitted.

[Valeriy Dmitriev via Phabricator via llvm-commits]

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

LG. Please fix indentation before pushing the patch. Thanks.



================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:9633
+        if (auto *PHI = dyn_cast<PHINode>(VecI))
+            Builder.SetInsertPoint(PHI->getParent()->getFirstNonPHI());
+        else
----------------
Indentation


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:9635
+        else
+            Builder.SetInsertPoint(VecI->getParent(),
+                                   std::next(VecI->getIterator()));
----------------
here too


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140580