[PATCH] D27696: [ThinLTO] Thin link efficiency: skip candidate added later with higher threshold (NFC)

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 12 20:22:52 PST 2016


tejohnson added a comment.

In https://reviews.llvm.org/D27696#620610, @mehdi_amini wrote:

> I'm surprised you're working on optimizing this, is this showing up on any profile?
>  Did you measure the actual benefit here?


Yes, see also https://reviews.llvm.org/D27687. I was running with a large app and also with the thresholds cranked up, and it was slowing down too much. I've shaved off at least 35% in that particular case from these improvements.


https://reviews.llvm.org/D27696





More information about the llvm-commits mailing list