OSDN Git Service

Export variable for device sanitize arch to Soong
authorColin Cross <ccross@android.com>
Wed, 2 Nov 2016 22:05:21 +0000 (15:05 -0700)
committerColin Cross <ccross@android.com>
Wed, 2 Nov 2016 22:20:25 +0000 (15:20 -0700)
Also renames the variable from SANITIZE_ARCH to SANITIZE_TARGET_ARCH,
and makes it only apply to the device.

Bug: 29498013
Test: No change to build.ninja files with m -j SANITIZE_TARGET=address
Change-Id: Ib5f6ab448f5d96d2426c983308136670f9a55b7b

core/config_sanitizers.mk
core/soong.mk

index 2e14fef..3a59ee3 100644 (file)
@@ -31,10 +31,12 @@ ifneq ($(SANITIZER.$(TARGET_PRODUCT).$(LOCAL_MODULE).CONFIG),)
   endif
 endif
 
-# Add a filter point for 32-bit vs 64-bit sanitization (to lighten the burden).
-SANITIZE_ARCH ?= 32 64
-ifeq ($(filter $(SANITIZE_ARCH),$(my_32_64_bit_suffix)),)
-  my_sanitize :=
+ifndef LOCAL_IS_HOST_MODULE
+  # Add a filter point for 32-bit vs 64-bit sanitization (to lighten the burden)
+  SANITIZE_TARGET_ARCH ?= $(TARGET_ARCH) $(TARGET_2ND_ARCH)
+  ifeq ($(filter $(SANITIZE_TARGET_ARCH),$(TARGET_$(LOCAL_2ND_ARCH_VAR_PREFIX)ARCH)),)
+    my_sanitize :=
+  endif
 endif
 
 # Add a filter point based on module owner (to lighten the burden). The format is a space- or
index 1d42eb0..73622a2 100644 (file)
@@ -48,6 +48,7 @@ $(SOONG_VARIABLES): FORCE
        echo '    "Allow_missing_dependencies": $(if $(TARGET_BUILD_APPS)$(filter true,$(SOONG_ALLOW_MISSING_DEPENDENCIES)),true,false),'; \
        echo '    "SanitizeHost": [$(if $(SANITIZE_HOST),"$(subst $(space),"$(comma)",$(SANITIZE_HOST))")],'; \
        echo '    "SanitizeDevice": [$(if $(SANITIZE_TARGET),"$(subst $(space),"$(comma)",$(SANITIZE_TARGET))")],'; \
+       echo '    "SanitizeDeviceArch": [$(if $(SANITIZE_TARGET_ARCH),"$(subst $(space),"$(comma)",$(SANITIZE_TARGET_ARCH))")],'; \
        echo '    "HostStaticBinaries": $(if $(strip $(BUILD_HOST_static)),true,false),'; \
        echo '    "Cpusets": $(if $(strip $(ENABLE_CPUSETS)),true,false),'; \
        echo '    "Schedboost": $(if $(strip $(ENABLE_SCHEDBOOST)),true,false),'; \