From bdb17f068c35c48370f3c3b7a564128d66c466ff Mon Sep 17 00:00:00 2001 From: Pirama Arumuga Nainar Date: Mon, 28 Aug 2017 21:50:17 -0700 Subject: [PATCH] Remove CFI-related WARs that seem no longer necessary Bug: http://b/33678192 Clang has been updated past the revisions mentioned in the work arounds. So these seem no longer necessary. Test: Build Change-Id: I08fee0db7c5785836a1ad0104688245435865cb7 --- cc/sanitize.go | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/cc/sanitize.go b/cc/sanitize.go index 9e1f02f7c..5d7f617e4 100644 --- a/cc/sanitize.go +++ b/cc/sanitize.go @@ -34,9 +34,8 @@ var ( cfiCflags = []string{"-flto", "-fsanitize-cfi-cross-dso", "-fsanitize-blacklist=external/compiler-rt/lib/cfi/cfi_blacklist.txt"} - // FIXME: revert the __cfi_check flag when clang is updated to r280031. cfiLdflags = []string{"-flto", "-fsanitize-cfi-cross-dso", "-fsanitize=cfi", - "-Wl,-plugin-opt,O1 -Wl,-export-dynamic-symbol=__cfi_check"} + "-Wl,-plugin-opt,O1"} cfiArflags = []string{"--plugin ${config.ClangBin}/../lib64/LLVMgold.so"} cfiExportsMapPath = "build/soong/cc/config/cfi_exports.map" cfiExportsMap android.Path @@ -364,9 +363,6 @@ func (sanitize *sanitize) flags(ctx ModuleContext, flags Flags) Flags { // __cfi_check needs to be built as Thumb (see the code in linker_cfi.cpp). LLVM is not set up // to do this on a function basis, so force Thumb on the entire module. flags.RequiredInstructionSet = "thumb" - // Workaround for b/33678192. CFI jumptables need Thumb2 codegen. Revert when - // Clang is updated past r290384. - flags.LdFlags = append(flags.LdFlags, "-march=armv7-a") } sanitizers = append(sanitizers, "cfi")