Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3)

Issue 2939923004: Disable ThinLTO optimizations on Linux with LLD. (Closed)

Created:
3 years, 6 months ago by pcc1
Modified:
3 years, 6 months ago
Reviewers:
Nico
CC:
chromium-reviews
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Disable ThinLTO optimizations on Linux with LLD. Unfortunately: - Binaries built with ThinLTO at -O2/--lto-O2 (which is what you currently get with use_thin_lto=true) are too large. - Binaries built with -Os introduce significant performance regressions. This issue most likely affects ThinLTO as well, at least according to local benchmarking. So that leaves -O2/--lto-O0, which is pretty much what we have now with full LTO. (With full LTO, --lto-O1 basically just means apply CFI, whole-program devirtualization, dead code elimination and CFG simplification.) Efforts have been made on the LLVM side to make ThinLTO at --lto-O0 generate binaries with as good quality as full LTO with --lto-O1 (simplifying the code generated for CFI checks (r304921) and allowing for more dead code elimination (r305482)). Now that those are complete, we are ready to start using --lto-O0 with ThinLTO. BUG=660216 R=thakis@chromium.org Review-Url: https://codereview.chromium.org/2939923004 Cr-Commit-Position: refs/heads/master@{#479875} Committed: https://chromium.googlesource.com/chromium/src/+/da072d13b2a1cc8459c29aa4e52584a4491a4ad6

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -0 lines) Patch
M build/config/compiler/BUILD.gn View 1 chunk +6 lines, -0 lines 0 comments Download

Messages

Total messages: 10 (6 generated)
pcc1
3 years, 6 months ago (2017-06-15 21:05:44 UTC) #1
Nico
lgtm
3 years, 6 months ago (2017-06-15 21:11:01 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2939923004/1
3 years, 6 months ago (2017-06-15 21:12:25 UTC) #7
commit-bot: I haz the power
3 years, 6 months ago (2017-06-15 22:38:55 UTC) #10
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/da072d13b2a1cc8459c29aa4e525...

Powered by Google App Engine
This is Rietveld 408576698