[PATCH] D45263: [SLP] Allow reordering of vectorization trees with reused instructions.

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added a comment.

In https://reviews.llvm.org/D45263#1060684, @RKSimon wrote:

> Would it be possible to increase test coverage?


Yes, I will try to add some more tests.



================
Comment at: lib/Transforms/Vectorize/SLPVectorizer.cpp:5785
+               "Order size must be the same as number of vectorized "
+               "instructions.");
         // TODO: reorder tree nodes without tree rebuilding.
----------------
RKSimon wrote:
> This assert doesn't seem to tally with the TODO just above?
Actually, it does, I just need to remove this TODO. This patch fixes this TODO.


Repository:
  rL LLVM

https://reviews.llvm.org/D45263