[PATCH] D27722: [GVNHoist] Move GVNHoist to function simplification part of pipeline.

Sebastian Pop via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 13 11:45:55 PST 2016

sebpop accepted this revision.
sebpop added a comment.

Overall LGTM.
In the past I have seen some improvements to function inlining due to hoisting happening before inlining.
Also there may be some sinking (in cfg-simplify) happening before we have a chance to hoist expressions.
Let's commit this, and I will report if I see perf degradations, in which case we may as well run hoisting before and after inlining.

Thanks for your patch!


More information about the llvm-commits mailing list