[PATCH] D20789: Consecutive memory access in Loop Vectorizer - fixed and simplified

Elena Demikhovsky via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 13 06:35:46 PDT 2016

delena added a comment.

In http://reviews.llvm.org/D20789#456016, @anemet wrote:

> In http://reviews.llvm.org/D20789#455936, @delena wrote:
> > I changed interface of getPtrStride(). Instead of one "Assume" parameter which means "additional run-time assumptions", I put 2 parameters - "Assume memory access versioning in run-time" and "Assume no-wrap in address calculation".
> >  isConsecutivePtr() sets to "true" only the first assumption.
> Why?  What's wrong with calling getPtrStirde with Assume=true?

2 tests were failing. "Assume-versioning"=true is ok for isConsecutivePtr(), but Assume-no-wrap should be "false".



More information about the llvm-commits mailing list