Merge "Revert "Disable dexpreopt for aosp_riscv64"" am: 1d3c4ae036

Original change: https://android-review.googlesource.com/c/platform/build/+/2511036

Change-Id: I7493acb1a6aa27d323e9b96ed60e38ac6c571230
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

core/board_config.mk

@@ -204,11 +204,8 @@ _board_strip_readonly_list += $(_build_broken_var_list) \
 
 # Conditional to building on linux, as dex2oat currently does not work on darwin.
 ifeq ($(HOST_OS),linux)
-  # TODO(riscv64) add compiler support and enable dexpreopt on RISC-V.
-  ifneq ($(TARGET_ARCH),riscv64)
   WITH_DEXPREOPT := true
 endif
-endif
 
 # ###############################################################
 # Broken build defaults

core/dex_preopt_config.mk

@@ -65,8 +65,6 @@ ifeq ($(HOST_OS),linux)
   # Non eng linux builds must have preopt enabled so that system server doesn't run as interpreter
   # only. b/74209329
   ifeq (,$(filter eng, $(TARGET_BUILD_VARIANT)))
-    # TODO(riscv64) add compiler support and enable dexpreopt on RISC-V.
-    ifeq (,$(filter riscv64, $(TARGET_ARCH)))
     ifneq (true,$(WITH_DEXPREOPT))
       ifneq (true,$(WITH_DEXPREOPT_BOOT_IMG_AND_SYSTEM_SERVER_ONLY))
         $(call pretty-error, DEXPREOPT must be enabled for user and userdebug builds)
@@ -74,7 +72,6 @@ ifeq ($(HOST_OS),linux)
     endif
   endif
 endif
-endif
 
 # Get value of a property. It is first searched from PRODUCT_VENDOR_PROPERTIES
 # and then falls back to PRODUCT_SYSTEM_PROPERTIES

target/board/generic_riscv64/BoardConfig.mk

@@ -26,6 +26,3 @@ include build/make/target/board/BoardConfigGsiCommon.mk
 
 # Temporary hack while prebuilt modules are missing riscv64.
 ALLOW_MISSING_DEPENDENCIES := true
-
-# Temporary until dex2oat works when targeting riscv64
-WITH_DEXPREOPT := false