[PATCH] D15720: [Polly] Use assumed context during ast generation

[Johannes Doerfert via llvm-commits]

jdoerfert added a subscriber: jdoerfert.
jdoerfert added a comment.

> Meinersbur added a comment.

> 

> Summary missing?

> 

> Could you explain why this changes anything, because the scheduler

>  should produce a schedule withing the assumed context only. (i.e.

>  because we gist the assumed context before scheduling if I am not

>  mistaken)


It does not. There is no assumed context involved in scheduling.

> Does this change the output other than adding conditionals? I guess

>  these are irrelevant because they are checked beforehand for loop

>  versioning anyway. Do the additional conditions affect execution time?


This removes dead statements from the schedule and hence the AST.

- F1231488: signature.asc <http://reviews.llvm.org/F1231488>


http://reviews.llvm.org/D15720