[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 23:27:52 PDT 2016


delena added a comment.

> Why were the tests failing?


When I turn on "Assume" parameter in the original version, I actually do two different things together:

1. Force check the predicated access
2. Disable wrap check in address calculation

When I do (1), I fix the original bug. But the two tests (we talked earlier) fail due to (2).
I separated (1) from (2) in the current patch and everything works fine.


Repository:
  rL LLVM

http://reviews.llvm.org/D20789





More information about the llvm-commits mailing list