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

AFDO: Do not mutate on disabled modules

Browse Source 
If a module is implicitly disabled due to compile_multilib: "64" on a
32-bit only device, we would insert a dependency to a non-existant AFDO
profile variation.

Test: presubmit
Bug: 350641338
Change-Id: I82851c4bc943405e48f7f35b3484fac2e509bfe5
This commit is contained in:
Yi Kong
2024-07-03 02:02:23 +09:00
parent dd274aba5f
commit f885689227
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
cc/afdo.go
View File

@@ -176,6 +176,9 @@ func (a *afdoTransitionMutator) IncomingTransition(ctx android.IncomingTransitio
func (a *afdoTransitionMutator) Mutate(ctx android.BottomUpMutatorContext, variation string) { func (a *afdoTransitionMutator) Mutate(ctx android.BottomUpMutatorContext, variation string) {
if m, ok := ctx.Module().(*Module); ok && m.afdo != nil { if m, ok := ctx.Module().(*Module); ok && m.afdo != nil {
if !m.Enabled(ctx) {
return
}
if variation == "" { if variation == "" {
// The empty variation is either a module that has enabled AFDO for itself, or the non-AFDO // The empty variation is either a module that has enabled AFDO for itself, or the non-AFDO
// variant of a dependency. // variant of a dependency.