OSDN Git Service

Disble CFI for ARM32 processes.
authorVishwath Mohan <vishwath@google.com>
Wed, 8 Feb 2017 04:28:07 +0000 (20:28 -0800)
committerVishwath Mohan <vishwath@google.com>
Thu, 9 Feb 2017 03:38:20 +0000 (19:38 -0800)
This CL disables CFI for 32-bit ARM processes, which is broken due to
a compiler error in the most recent version of clang.

Bug: 35157333
Test: ENABLE_CFI=true m -j40 does not enable CFI for 32-bit processes
Change-Id: I6a6e1d14c8365da1056b4cbccad11ef2f15c70b2

core/config_sanitizers.mk

index 5eaf7b1..57a7993 100644 (file)
@@ -68,6 +68,12 @@ ifeq ($(strip $(ENABLE_CFI)),)
   my_sanitize_diag := $(filter-out cfi,$(my_sanitize_diag))
 endif
 
+# Disable CFI for arm32 (b/35157333).
+ifneq ($(filter arm,$(TARGET_$(LOCAL_2ND_ARCH_VAR_PREFIX)ARCH)),)
+  my_sanitize := $(filter-out cfi,$(my_sanitize))
+  my_sanitize_diag := $(filter-out cfi,$(my_sanitize_diag))
+endif
+
 # CFI needs gold linker, and mips toolchain does not have one.
 ifneq ($(filter mips mips64,$(TARGET_$(LOCAL_2ND_ARCH_VAR_PREFIX)ARCH)),)
   my_sanitize := $(filter-out cfi,$(my_sanitize))