StrixOS/build_soong
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Do not enable global ThinLTO for tests

Browse Source 
Tests are not shipped, no need to spend extra CPU cycles optimising them
during build.

Test: GLOBAL_THINLTO=true m
Bug: 169004486
Change-Id: I66ede9c01d43b574a7fe9f74f0bc6ba97f51be06
This commit is contained in:
Yi Kong
2022-09-06 16:24:00 +08:00
parent 0228d27943
commit 56fc1b6902
2 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
cc/cc.go
View File

@@ -485,6 +485,7 @@ type ModuleContextIntf interface {
static() bool static() bool
staticBinary() bool staticBinary() bool
testBinary() bool testBinary() bool
testLibrary() bool
header() bool header() bool
binary() bool binary() bool
object() bool object() bool
@@ -1486,6 +1487,10 @@ func (ctx *moduleContextImpl) testBinary() bool {
return ctx.mod.testBinary() return ctx.mod.testBinary()
} }
func (ctx *moduleContextImpl) testLibrary() bool {
return ctx.mod.testLibrary()
}
func (ctx *moduleContextImpl) header() bool { func (ctx *moduleContextImpl) header() bool {
return ctx.mod.Header() return ctx.mod.Header()
} }

5
cc/lto.go
View File

@@ -140,12 +140,13 @@ func (lto *lto) DefaultThinLTO(ctx BaseModuleContext) bool {
lib32 := ctx.Arch().ArchType.Multilib == "lib32" lib32 := ctx.Arch().ArchType.Multilib == "lib32"
// CFI enables full LTO. // CFI enables full LTO.
cfi := ctx.isCfi() cfi := ctx.isCfi()
// Performance and binary size are less important for host binaries. // Performance and binary size are less important for host binaries and tests.
host := ctx.Host() host := ctx.Host()
test := ctx.testBinary() || ctx.testLibrary()
// FIXME: ThinLTO for VNDK produces different output. // FIXME: ThinLTO for VNDK produces different output.
// b/169217596 // b/169217596
vndk := ctx.isVndk() vndk := ctx.isVndk()
return GlobalThinLTO(ctx) && !lto.Never() && !lib32 && !cfi && !host && !vndk return GlobalThinLTO(ctx) && !lto.Never() && !lib32 && !cfi && !host && !test && !vndk
} }
func (lto *lto) FullLTO() bool { func (lto *lto) FullLTO() bool {