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

[Chandler Carruth via Phabricator via llvm-commits]

chandlerc added a comment.

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?

For example, we can easily get test suite results just using `llvm-lit` directly with a build of the test-suite. Is there anything more comparable to this in terms of overhead for developers?

(Also, your instructions talk about `target-arm64-iphoneos` which I suspect needs to be adjusted for most folks?)

> This will test the given compiler and generate a report in report.json in the sandbox folder with the results. There is a `test-suite/utils/compare.py` script for comparing such reports, though it has additional requirements for python packages, so it might be a bit cumbersome to use it.

Heh, I don't find compare.py cumbersome, doing the full LNT setup seems much moreso... Installing packages is relatively easy, its the steps and deep layers that seem somewhat burdensome (IMO). I've really like the direction of the test-suite recently to run with llvm-lit and nothing else really.


https://reviews.llvm.org/D38154