[PATCH] D38154: [PassManager] Run global opts after the inliner
Davide Italiano via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Oct 1 22:50:21 PDT 2017
davide added a comment.
Basically, thanks Chandler.
I'm more than happy to make this an RFT (although I'm not holding my breath :)) and wait a bit before committing.
FWIW, I expect this to be fairly reasonable to do nonetheless (in fact, FWIW, the full LTO pipeline does it, and it never seemed to have a huge impact, or at least I haven't seen this showing up a lot in a profile). I'll update the new patch tomorrow and try to run some tests (sorry, this one doesn't even compile as I rewrote instead of copying from the machine where I ran the tests).
https://reviews.llvm.org/D38154
More information about the llvm-commits
mailing list