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

Johannes Doerfert via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 22 17:22:05 PST 2015

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>


More information about the llvm-commits mailing list