[PATCH] D13487: [Polly] Load/Store scalar accesses before/after the statement itself

Tobias Grosser via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 17 11:39:46 PDT 2015


grosser added a comment.

> Meinersbur added a comment.

> 

> In http://reviews.llvm.org/D13487#269745, @grosser wrote:

> 

> > OK. Maybe add a test case that would have failed with the code I proposed.

> 

> 

> Isl/CodeGen/phi_in_exit_early_lnt_failure_2.ll already fails with that code (just tried it). Do you want a dedicated test case?


No, I just did not realize it would fail. (It did not in my earlier 
experiments, but does now). Then its fine as it is.

Best,
Tobias


http://reviews.llvm.org/D13487





More information about the llvm-commits mailing list