[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