[PATCH] D16223: Update to ISL 0.16.1

Michael Kruse via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 15 08:07:05 PST 2016


Meinersbur accepted this revision.
Meinersbur added a reviewer: Meinersbur.
Meinersbur added a comment.
This revision is now accepted and ready to land.

In http://reviews.llvm.org/D16223#327874, @grosser wrote:

> LGTM.
>
> The difference is because the isl generated code has changed
> from:
>
>   for (int c0 = 0; c0 < floord(tmp, 2) - 100; c0 += 1)
>     Stmt_stmt_P(c0);
>   if (tmp <= 201)
>     Stmt_stmt_P(0);
>   
>
> to:
>
>   Stmt_stmt_P(0);
>   for (int c0 = 1; c0 < floord(tmp, 2) - 100; c0 += 1)
>     Stmt_stmt_P(c0);
>   
>
> The instruction that has move is generated before the loop. In
>  the old code, this is polly.start. In the new code, this is 
>  polly.Stmt.stmt.P.
>
> Thank you for updating isl so quickly!
>
> Best,
> Tobias


Thanks. I thought it had to do with invariant loads, because of the file name and polly.start is not from a statement.


http://reviews.llvm.org/D16223





More information about the llvm-commits mailing list