[PATCH] D50143: [SLP] Fix insert point for reused extract instructions.

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

This version makes a lot more sense. :)



================
Comment at: lib/Transforms/Vectorize/SLPVectorizer.cpp:3112
           // TODO: Merge this shuffle with the ReorderShuffleMask.
-          if (!E->ReorderIndices.empty())
+          if (E->ReorderIndices.empty())
             Builder.SetInsertPoint(VL0);
----------------
Please add a test for the case where `NeedToShuffleReuses` is true and `E->ReorderIndices.empty()` is false.


Repository:
  rL LLVM

https://reviews.llvm.org/D50143