Merge "Support instrumenting all PGO-enabled modules"

This commit is contained in:
Treehugger Robot
2018-01-26 02:52:44 +00:00
committed by Gerrit Code Review

View File

@@ -197,7 +197,7 @@ func (pgo *pgo) begin(ctx BaseModuleContext) {
} }
// This module should be instrumented if ANDROID_PGO_INSTRUMENT is set // This module should be instrumented if ANDROID_PGO_INSTRUMENT is set
// and includes a benchmark listed for this module // and includes 'all', 'ALL' or a benchmark listed for this module.
// //
// TODO Validate that each benchmark instruments at least one module // TODO Validate that each benchmark instruments at least one module
pgo.Properties.ShouldProfileModule = false pgo.Properties.ShouldProfileModule = false
@@ -207,12 +207,16 @@ func (pgo *pgo) begin(ctx BaseModuleContext) {
pgoBenchmarksMap[b] = true pgoBenchmarksMap[b] = true
} }
if pgoBenchmarksMap["all"] == true || pgoBenchmarksMap["ALL"] == true {
pgo.Properties.ShouldProfileModule = true
} else {
for _, b := range pgo.Properties.Pgo.Benchmarks { for _, b := range pgo.Properties.Pgo.Benchmarks {
if pgoBenchmarksMap[b] == true { if pgoBenchmarksMap[b] == true {
pgo.Properties.ShouldProfileModule = true pgo.Properties.ShouldProfileModule = true
break break
} }
} }
}
} }
func (pgo *pgo) deps(ctx BaseModuleContext, deps Deps) Deps { func (pgo *pgo) deps(ctx BaseModuleContext, deps Deps) Deps {