[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