[PATCH] D12765: [LV] Allow vectorization of loops with induction post-inc expressions

Fri Sep 18 19:00:36 PDT 2015

hfinkel added a comment.

In http://reviews.llvm.org/D12765#249249, @mzolotukhin wrote:

> 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.

Great, thanks!

> 

> 

> > 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.

I doubt it is complicated enough ;) -- but that does not necessarily mean it needs this enhancement.

> Thanks,

> Michael

Repository:
  rL LLVM

http://reviews.llvm.org/D12765