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

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Apr 7 06:52:49 PDT 2018


RKSimon added a comment.

Would it be possible to increase test coverage?



================
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.
----------------
This assert doesn't seem to tally with the TODO just above?


Repository:
  rL LLVM

https://reviews.llvm.org/D45263





More information about the llvm-commits mailing list