Merge "Limit LTO inlining even when profile is present" into main

This commit is contained in:
Treehugger Robot
2023-08-02 22:27:04 +00:00
committed by Gerrit Code Review

View File

@@ -135,10 +135,14 @@ func (lto *lto) flags(ctx BaseModuleContext, flags Flags) Flags {
ltoLdFlags = append(ltoLdFlags, cachePolicyFormat+policy)
}
// If the module does not have a profile, be conservative and limit cross TU inline
// limit to 5 LLVM IR instructions, to balance binary size increase and performance.
if !ctx.Darwin() && !ctx.isPgoCompile() && !ctx.isAfdoCompile() {
ltoLdFlags = append(ltoLdFlags, "-Wl,-plugin-opt,-import-instr-limit=5")
// Reduce the inlining threshold for a better balance of binary size and
// performance.
if !ctx.Darwin() {
if ctx.isPgoCompile() || ctx.isAfdoCompile() {
ltoLdFlags = append(ltoLdFlags, "-Wl,-plugin-opt,-import-instr-limit=40")
} else {
ltoLdFlags = append(ltoLdFlags, "-Wl,-plugin-opt,-import-instr-limit=5")
}
}
flags.Local.CFlags = append(flags.Local.CFlags, ltoCFlags...)