tequilaOS/platform_build_soong
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Merge "Use --dependency-file when linking elf files." am: ff22afa704 am: 87089903c2

Browse source 
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1975926

Change-Id: I1413949241ed383368027fb3f1a10cb60cfb74f5
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Treehugger Robot 2022-04-29 18:09:12 +00:00 committed by Automerger Merge Worker
commit 0b80b69378
1 changed files with 30 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

32
cc/builder.go
View file

@ -853,13 +853,24 @@ func transformObjToDynamicBinary(ctx android.ModuleContext,
deps = append(deps, crtBegin...)
deps = append(deps, crtEnd...)
var depFile android.WritablePath
var depFileLdFlags string
depsType := blueprint.DepsNone
if !ctx.Windows() && !ctx.Darwin() {
// lld only supports --dependency-file for elf files
depFile = outputFile.ReplaceExtension(ctx, "d")
depFileLdFlags = " -Wl,--dependency-file=" + depFile.String()
depsType = blueprint.DepsGCC
implicitOutputs = append(implicitOutputs, depFile)
}
rule := ld
args := map[string]string{
"ldCmd": ldCmd,
"crtBegin": strings.Join(crtBegin.Strings(), " "),
"libFlags": strings.Join(libFlagsList, " "),
"extraLibFlags": flags.extraLibFlags,
"ldFlags": flags.globalLdFlags + " " + flags.localLdFlags,
"ldFlags": flags.globalLdFlags + " " + flags.localLdFlags + depFileLdFlags,
"crtEnd": strings.Join(crtEnd.Strings(), " "),
}
if ctx.Config().UseRBE() && ctx.Config().IsEnvTrue("RBE_CXX_LINKS") {
@ -870,6 +881,8 @@ func transformObjToDynamicBinary(ctx android.ModuleContext,
ctx.Build(pctx, android.BuildParams{
Rule: rule,
Deps: depsType,
Depfile: depFile,
Description: "link " + outputFile.Base(),
Output: outputFile,
ImplicitOutputs: implicitOutputs,
@ -1023,18 +1036,33 @@ func transformObjsToObj(ctx android.ModuleContext, objFiles android.Paths,
ldCmd := "${config.ClangBin}/clang++"
var implicitOutputs android.WritablePaths
var depFile android.WritablePath
var depFileLdFlags string
depsType := blueprint.DepsNone
if !ctx.Windows() && !ctx.Darwin() {
// lld only supports --dependency-file for elf files
depFile = outputFile.ReplaceExtension(ctx, "d")
depFileLdFlags = " -Wl,--dependency-file=" + depFile.String()
depsType = blueprint.DepsGCC
implicitOutputs = append(implicitOutputs, depFile)
}
rule := partialLd
args := map[string]string{
"ldCmd": ldCmd,
"ldFlags": flags.globalLdFlags + " " + flags.localLdFlags,
"ldFlags": flags.globalLdFlags + " " + flags.localLdFlags + depFileLdFlags,
}
if ctx.Config().UseRBE() && ctx.Config().IsEnvTrue("RBE_CXX_LINKS") {
rule = partialLdRE
args["inCommaList"] = strings.Join(objFiles.Strings(), ",")
args["implicitInputs"] = strings.Join(deps.Strings(), ",")
args["implicitOutputs"] = strings.Join(implicitOutputs.Strings(), ",")
}
ctx.Build(pctx, android.BuildParams{
Rule: rule,
Deps: depsType,
Depfile: depFile,
Description: "link " + outputFile.Base(),
Output: outputFile,
Inputs: objFiles,