commit | e1a1b9259b01279305ce351100bc975d54a7829f | [log] [tgz] |
---|---|---|
author | krasin <krasin@chromium.org> | Tue Apr 25 00:42:01 2017 |
committer | Commit bot <commit-bot@chromium.org> | Tue Apr 25 00:42:01 2017 |
tree | 8db63cc607edcd6bdcfe2f77c25887684e84b5cf | |
parent | 3037dcde1b6afbe14e3d8e63daf1b7fe45b50f5b [diff] |
Revert of Enable ThinLTO and LLD for POSIX LTO by default on Linux. (patchset #4 id:60001 of https://codereview.chromium.org/2831213006/ ) Reason for revert: Code size increase: https://chromeperf.appspot.com/report?sid=7826328365affe1842778994c934cbe08c7f9b78ef5d8895ec85d3094bb8972d&start_rev=466555&end_rev=466782 Postmortem is coming. TL;DR: worse dead code elimination in ThinLTO Original issue's description: > Enable ThinLTO and LLD for POSIX LTO by default on Linux. > > Eventually, we will enable LLD on Linux even for regular > builds, but it's natural to make incremental steps here. > > ThinLTO brings multi-threaded linking for LinkTimeOptimization > builds, which allows to speed up codegen considerably (up to 4x). > ThinLTO also manages a cache inside out/<gn-config>/thinlto-cache > directory, that improves incremental linking as well. > > There's a cache pruning policy that will prevent the cache from > growing indefinitely. The policy is not yet finalized, and > complaints / suggestions are welcome. > > BUG=660216,607968 > > Review-Url: https://codereview.chromium.org/2831213006 > Cr-Commit-Position: refs/heads/master@{#466732} > Committed: https://chromium.googlesource.com/chromium/src/+/7997bbe88aef7eeca37111dcb8426f0430347fb5 TBR=dpranke@chromium.org,thakis@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=660216,607968 Review-Url: https://codereview.chromium.org/2842683002 Cr-Original-Commit-Position: refs/heads/master@{#466842} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: 6aee4d67dcc67850c8ec7b9fc181cb7d91b55734