[PATCH] D38154: [PassManager] Run global opts after the inliner

[Davide Italiano via Phabricator via llvm-commits]

davide added a comment.

I uploaded the results before and after. There are some performance wins in terms of size, which is good (this is at `-Os`).
The only concerning regression is 7zip which gets around 1.8% slower (compile time wise) for  some amount of win (reduction in text and eh_frame/eh_frame_hdr).
I ran that several times and I saw this value oscillating between 0.8% and 1.8% so there's some noise to take in account.
Overall, I think this change didn't significantly regressed anywhere so I'd vote for going forward (I'll also send an heads-up on llvm-dev, just in case).
Thoughts?

Thanks!


https://reviews.llvm.org/D38154