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

[Alexey Bataev via Phabricator via llvm-commits]

ABataev 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());
----------------
dtemirbulatov wrote:
> 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.
Could you explain why?


https://reviews.llvm.org/D28907