[PATCH] D28996: [PM] Flesh out the new pm LTO pipeline

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 22 16:48:36 PST 2017

davide added a comment.

In https://reviews.llvm.org/D28996#652797, @mehdi_amini wrote:

> What is the justification instead of adopting the ThinLTO pipeline?

The two problems are largely orthogonal. This is a port of the old LTO pipeline to the new pass manager. The aim is to make sure the current LTO pipeline works correctly with the new pass manager. I prefer to keep the old functionality in the new pass manager until it stabilizes. After that (or in parallel), we can decide whether we want to unify the two pipelines (if it's possible/reasonable). 
On top of that, currently plugging the ThinLTO pipeline for LTO results in 2x/2.5x increase in compile time. Definitely not an hit I want to pay.



More information about the llvm-commits mailing list