[PATCH] D21773: [clang] Update an optimization remark test for change D18777

Adam Nemet via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 28 21:43:26 PDT 2016


anemet added a comment.

In http://reviews.llvm.org/D21773#469596, @lihuang wrote:

> IV is promoted to 64-bit but the trunc/zext cannot be eliminated (at least cannot be eliminated with the -O1 pass pipeline). Then optimzation remark becomes:
>
>   optimization-remark-options.c:17:3: remark: loop not vectorized: cannot identify array bounds
>      [-Rpass-analysis=loop-vectorize]
>   for (int i = 0; i < N; i++) {


That sounds like an optimization regression.  It seems to me that you could create a testcase with fewer arrays than in the above test such that you don't exceed the max number of memchecks.  This new testcase would be vectorized before http://reviews.llvm.org/D18777 but not after.

Adam


http://reviews.llvm.org/D21773





More information about the cfe-commits mailing list