[PATCH] D12264: Identify and hoist loop invariant loads

[Johannes Doerfert via llvm-commits]

jdoerfert added a comment.

> I did not yet look at this patch in detail, but here already one test case that

>  I wanted to handle and which does not seem to be resolved by your patch (intentionally?).


A similar problem is already included as a test case, see http://reviews.llvm.org/D12264#e3d3ea83 .

> This test case still fails with SCEV not being able to derive the number of loop

>  iterations.


That is a different problem not tackled here. Once we have isl based domain construction this should work ;) (if there are nor bugs obviously)


http://reviews.llvm.org/D12264