[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