[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