diff --git a/cc/builder.go b/cc/builder.go index 064613248..279c1da7e 100644 --- a/cc/builder.go +++ b/cc/builder.go @@ -256,6 +256,7 @@ type builderFlags struct { systemIncludeFlags string groupStaticLibs bool + arGoldPlugin bool stripKeepSymbols bool stripKeepMiniDebugInfo bool @@ -512,6 +513,9 @@ func TransformObjToStaticLib(ctx android.ModuleContext, objFiles android.Paths, if !ctx.Darwin() { arFlags += " -format=gnu" } + if flags.arGoldPlugin { + arFlags += " --plugin ${config.LLVMGoldPlugin}" + } if flags.arFlags != "" { arFlags += " " + flags.arFlags } diff --git a/cc/cc.go b/cc/cc.go index 9cc7dfa38..49fefe965 100644 --- a/cc/cc.go +++ b/cc/cc.go @@ -142,6 +142,7 @@ type Flags struct { LdFlagsDeps android.Paths // Files depended on by linker flags GroupStaticLibs bool + ArGoldPlugin bool // Whether LLVM gold plugin option is passed to llvm-ar } type ObjectLinkerProperties struct { diff --git a/cc/lto.go b/cc/lto.go index 91b11b539..4757fc7e8 100644 --- a/cc/lto.go +++ b/cc/lto.go @@ -86,7 +86,7 @@ func (lto *lto) flags(ctx BaseModuleContext, flags Flags) Flags { // https://github.com/android-ndk/ndk/issues/498. flags.LdFlags = append(flags.LdFlags, "-Wl,-plugin-opt,-emulated-tls") } - flags.ArFlags = append(flags.ArFlags, " --plugin ${config.LLVMGoldPlugin}") + flags.ArGoldPlugin = true } return flags } diff --git a/cc/sanitize.go b/cc/sanitize.go index ac6cb778a..c47c31948 100644 --- a/cc/sanitize.go +++ b/cc/sanitize.go @@ -38,7 +38,6 @@ var ( "-fsanitize-blacklist=external/compiler-rt/lib/cfi/cfi_blacklist.txt"} cfiLdflags = []string{"-flto", "-fsanitize-cfi-cross-dso", "-fsanitize=cfi", "-Wl,-plugin-opt,O1"} - cfiArflags = []string{"--plugin ${config.ClangBin}/../lib64/LLVMgold.so"} cfiExportsMapPath = "build/soong/cc/config/cfi_exports.map" cfiExportsMap android.Path cfiStaticLibsMutex sync.Mutex @@ -407,7 +406,7 @@ func (sanitize *sanitize) flags(ctx ModuleContext, flags Flags) Flags { // See b/72706604 or https://github.com/android-ndk/ndk/issues/498. flags.LdFlags = append(flags.LdFlags, "-Wl,-plugin-opt,-emulated-tls") } - flags.ArFlags = append(flags.ArFlags, cfiArflags...) + flags.ArGoldPlugin = true if Bool(sanitize.Properties.Sanitize.Diag.Cfi) { diagSanitizers = append(diagSanitizers, "cfi") } diff --git a/cc/util.go b/cc/util.go index 5131b09a9..92a32bc9a 100644 --- a/cc/util.go +++ b/cc/util.go @@ -84,6 +84,7 @@ func flagsToBuilderFlags(in Flags) builderFlags { systemIncludeFlags: strings.Join(in.SystemIncludeFlags, " "), groupStaticLibs: in.GroupStaticLibs, + arGoldPlugin: in.ArGoldPlugin, } }