From 9fa025491da8c6493d9286a7d4980105b649406e Mon Sep 17 00:00:00 2001 From: Calin Juravle Date: Fri, 25 Mar 2016 17:29:15 +0000 Subject: [PATCH] Don't check if forward locked apps are used by others Forward locked apps don't share their code so there is no point moving from profile guided to full compilation. Bug: 27334750 Bug: 26080105 Change-Id: I195d48504435bc66e890470f04056c4279d0bb2f --- .../core/java/com/android/server/pm/PackageDexOptimizer.java | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/services/core/java/com/android/server/pm/PackageDexOptimizer.java b/services/core/java/com/android/server/pm/PackageDexOptimizer.java index 5ceb65fb81fad..4ca615dd55c9a 100644 --- a/services/core/java/com/android/server/pm/PackageDexOptimizer.java +++ b/services/core/java/com/android/server/pm/PackageDexOptimizer.java @@ -138,9 +138,8 @@ class PackageDexOptimizer { boolean isProfileGuidedFilter = DexFile.isProfileGuidedCompilerFilter(targetCompilerFilter); // If any part of the app is used by other apps, we cannot use profile-guided // compilation. - // TODO: This needs to be refactored to be also checked when the target mode is - // profile-guided. - if (isProfileGuidedFilter) { + // Skip the check for forward locked packages since they don't share their code. + if (isProfileGuidedFilter && !pkg.isForwardLocked()) { for (String path : paths) { if (isUsedByOtherApps(path)) { checkProfiles = false;