[PATCH] D21237: LoopRotation: remove iterative calls to rotateLoops

[Sebastian Pop via llvm-commits]

sebpop added a comment.

In http://reviews.llvm.org/D21237#457089, @atrick wrote:

> When two passes share common transformations, they should share a utility. They should not be combined into one pass unless they 
>  (a) are driven be the same analysis
>  (b) have the same pass ordering dependencies


I stand corrected: loop-rotation and jump-thread should remain in separate passes.

Maybe we can improve the current code by using the same code generation functionality:
I see that currently there is no function to duplicate a single entry multiple exits (SEME) path
needed for both loop-rotation and jump-threading.


http://reviews.llvm.org/D21237