diff options
author | Chungro Lee <chungro@google.com> | 2024-03-28 01:13:59 +0000 |
---|---|---|
committer | Priyanka Advani <padvani@google.com> | 2024-03-28 01:25:42 +0000 |
commit | 97f68587fff22676d3f71b7e0ea7f2d2c4649558 (patch) | |
tree | 660e085a9ec8f44ad199008d135cb7668d1cff2d | |
parent | 7549990a957c66d1ed29bde27da05be001491f9f (diff) | |
download | soong-97f68587fff22676d3f71b7e0ea7f2d2c4649558.tar.gz |
Revert^3 "Enable full LTO optimization by default"
7549990a957c66d1ed29bde27da05be001491f9f
Reason for revert: b/331528903
Change-Id: Idb013e97c84aeaf969522dc77abf94f23f721181
-rw-r--r-- | cc/lto.go | 11 |
1 files changed, 6 insertions, 5 deletions
@@ -54,9 +54,6 @@ type LTOProperties struct { // Use -fwhole-program-vtables cflag. Whole_program_vtables *bool - - // Use --lto-O0 flag. - Lto_O0 *bool } type lto struct { @@ -113,8 +110,12 @@ func (lto *lto) flags(ctx ModuleContext, flags Flags) Flags { ltoCFlags := []string{"-flto=thin", "-fsplit-lto-unit"} var ltoLdFlags []string - // Do not perform costly LTO optimizations for Eng builds. - if Bool(lto.Properties.Lto_O0) || ctx.Config().Eng() { + // The module did not explicitly turn on LTO. Only leverage LTO's + // better dead code elimination and CFG simplification, but do + // not perform costly optimizations for a balance between compile + // time, binary size and performance. + // Apply the same for Eng builds as well. + if !lto.ThinLTO() || ctx.Config().Eng() { ltoLdFlags = append(ltoLdFlags, "-Wl,--lto-O0") } |