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

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added a comment.

In https://reviews.llvm.org/D38154#886712, @chandlerc wrote:

> In https://reviews.llvm.org/D38154#886708, @mzolotukhin wrote:
>
> > I'd recommend using LNT,
>
>
> This seems like a really cumbersome process to expect folks to go through for changes intending to improve compile time. Is there no way to simplify it?


Which part is cumbersome? I use to do this all the time, and it is actually pretty nice when you get use to it. Especially it can build once and run multiple time each test to get more sample and reduce the measurement noise. The comparison script is easy to use as well.

Note, in case there is any confusion: this does not require to setup or use an LNT *instance* (as in the web application).

On the other hand I never found any similar convenience with the cmake/lit use of the test-suite (maybe too young to have proper support?).


https://reviews.llvm.org/D38154