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

Fix speed-profile compilation for prebuilds

Browse source 
We were testing the wrong variable when setting the dex2oat flags

Test: m; flash, dumpsys dexopt and check for speed-profile
Bug: 78515726

(cherry picked from commit 7d701355e9)

Merged-In: I57c0e11aab2e4879c3ba3b2851f356b5864f5da9
Change-Id: I2e2f9da1dc9ed71977ec1d698433674c731f78bc
This commit is contained in:
Calin Juravle 2018-04-25 15:25:46 -07:00
parent 8faa2ec0d8
commit 62c2893849
1 changed files with 7 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
core/dex_preopt_odex_install.mk
View file

@ -99,10 +99,13 @@ LOCAL_DEX_PREOPT_GENERATE_PROFILE := false
endif endif
ifndef LOCAL_DEX_PREOPT_GENERATE_PROFILE ifndef LOCAL_DEX_PREOPT_GENERATE_PROFILE
# If LOCAL_DEX_PREOPT_GENERATE_PROFILE is not defined, default it based on the existence of the # If LOCAL_DEX_PREOPT_GENERATE_PROFILE is not defined, default it based on the existence of the
# profile class listing. TODO: Use product specific directory here. # profile class listing. TODO: Use product specific directory here.
my_classes_directory := $(PRODUCT_DEX_PREOPT_PROFILE_DIR) my_classes_directory := $(PRODUCT_DEX_PREOPT_PROFILE_DIR)
LOCAL_DEX_PREOPT_PROFILE := $(my_classes_directory)/$(LOCAL_MODULE).prof LOCAL_DEX_PREOPT_PROFILE := $(my_classes_directory)/$(LOCAL_MODULE).prof
ifneq (,$(wildcard $(LOCAL_DEX_PREOPT_PROFILE))) ifneq (,$(wildcard $(LOCAL_DEX_PREOPT_PROFILE)))
my_process_profile := true my_process_profile := true
my_profile_is_text_listing := false my_profile_is_text_listing := false
@ -162,7 +165,6 @@ $(my_built_profile):
|| echo "Profile out of date for $(PRIVATE_BUILT_MODULE)" || echo "Profile out of date for $(PRIVATE_BUILT_MODULE)"
endif endif
my_process_profile :=
my_profile_is_text_listing := my_profile_is_text_listing :=
dex_preopt_profile_src_file := dex_preopt_profile_src_file :=
@ -233,7 +235,7 @@ installed_vdex := $(strip $(installed_vdex))
installed_art := $(strip $(installed_art)) installed_art := $(strip $(installed_art))
ifdef built_odex ifdef built_odex
ifeq (true,$(LOCAL_DEX_PREOPT_GENERATE_PROFILE)) ifeq (true,$(my_process_profile))
$(built_odex): $(my_built_profile) $(built_odex): $(my_built_profile)
$(built_odex): PRIVATE_PROFILE_PREOPT_FLAGS := --profile-file=$(my_built_profile) $(built_odex): PRIVATE_PROFILE_PREOPT_FLAGS := --profile-file=$(my_built_profile)
else else
@ -268,7 +270,7 @@ ifeq (,$(filter --compiler-filter=%, $(LOCAL_DEX_PREOPT_FLAGS)))
# 'speed' compiler filter. # 'speed' compiler filter.
LOCAL_DEX_PREOPT_FLAGS += --compiler-filter=speed LOCAL_DEX_PREOPT_FLAGS += --compiler-filter=speed
else else
ifeq (true,$(LOCAL_DEX_PREOPT_GENERATE_PROFILE)) ifeq (true,$(my_process_profile))
# For non system server jars, use speed-profile when we have a profile. # For non system server jars, use speed-profile when we have a profile.
LOCAL_DEX_PREOPT_FLAGS += --compiler-filter=speed-profile LOCAL_DEX_PREOPT_FLAGS += --compiler-filter=speed-profile
else else
@ -388,4 +390,6 @@ endif # LOCAL_DEX_PREOPT
ALL_MODULES.$(my_register_name).INSTALLED += $(my_installed_profile) ALL_MODULES.$(my_register_name).INSTALLED += $(my_installed_profile)
ALL_MODULES.$(my_register_name).BUILT_INSTALLED += $(build_installed_profile) ALL_MODULES.$(my_register_name).BUILT_INSTALLED += $(build_installed_profile)
my_process_profile :=
$(my_all_targets): $(my_installed_profile) $(my_all_targets): $(my_installed_profile)