[PATCH] D81416: [LV][SLP] Interleave to expose ILP for small loops with scalar reductions.
Aaron H Liu via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 30 09:12:21 PDT 2020
AaronLiu added a comment.
In this patch we "Interleave to expose ILP". The whole purpose of this patch is to "expose ILP", and the approach is to "Interleave".
- I worry about that if we remove those testcases, no vectorization(parallelism) results due to this patch can be seen, and people will have no idea where do we "expose ILP"?
- We have ever discussed that what @spatel suggested adding two testcases under PhaseOrdering to show the baseline results with the option in this patch on and off "do make sense".
- The purpose of adding lit tests is to catch future regressions, i.e., if someone make the vectorization **not** working, we will catch it with the testcases we added.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D81416/new/
https://reviews.llvm.org/D81416
More information about the llvm-commits
mailing list