[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.
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