[PATCH] D28907: [SLP] Fix for PR30787: Failure to beneficially vectorize 'copyable' elements in integer binary ops.

[Dinar Temirbulatov via Phabricator via llvm-commits]

dtemirbulatov added inline comments.


================
Comment at: lib/Transforms/Vectorize/SLPVectorizer.cpp:3112
+      int MaxDist = 0;
+      while (Bundle) {
+        int Dist = std::distance(BB->begin(), Bundle->Inst->getIterator());
----------------
ABataev wrote:
> Why you can't put bundles in the list in the right order: from the very first instruction to the very last?
I could do this in scheduleBlock() function with a queue, but that could add additional complexity.


https://reviews.llvm.org/D28907