[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 18 18:19:01 PDT 2015


mzolotukhin added a comment.

Hi Hal,

> I agree that this is a good thing to do. Can you post the patch for this? I'd like to get Sanjoy to look at this.


Actually, Sanjoy has already commented on the patch I posted above on the mailing list, but it didn't get to phabricator for some reason. I can create a separate phab-review for this patch, but I'd like to run some testing before that, which I hope I can do next week.

> That having been said, do you have a theoretical argument that IndVarSimplify can always fix this?


Nope, I can't prove that. However, if this will fix all the cases we care about, then I'd rather not add new entities to vectorizer - it's already complicated enough.

Thanks,
Michael


Repository:
  rL LLVM

http://reviews.llvm.org/D12765





More information about the llvm-commits mailing list