[PATCH] D12765: [LV] Allow vectorization of loops with induction post-inc expressions
Michael Zolotukhin via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 11 16:04:18 PDT 2015
mzolotukhin added a comment.
Hi Jakub,
I was able to reproduce the original issue with the test you posted, thanks! However, it still looks like the problem isn't in vectorizer. E.g. if you replace `int n = y < BUFF_SIZE ? (y - 1) : (BUFF_SIZE - 1)` (which is equivalent to the expression in your test) with something else, like `int n = y` or `int n = y*7%19`, the loop would be vectorized. Looking at dumps, I see that earlier passes (starting from MergedLoadStoreMotion) behave differently with these value of upper bound, which doesn't look right, and we should fix those passes, not vectorizer.
Thanks,
Michael
Repository:
rL LLVM
http://reviews.llvm.org/D12765
More information about the llvm-commits
mailing list