From ee7e22c6435048f1b37db4d944e3b6f4c65a77e5 Mon Sep 17 00:00:00 2001 From: Pirama Arumuga Nainar Date: Thu, 20 Feb 2020 15:08:30 -0800 Subject: Change behavior of doubleBuildWithWNoError for Android "Double build with -Wno-error" is always on in the Android useLlvmNext wrapper and always off in the production wrapper. The warnings JSON is written to $OUT_DIR/warnings_reports. NFC for the ChromeOS configs. Originally reviewed in https://android-review.googlesource.com/c/1242126. BUG=b:149836702 TEST=go test Change-Id: I5f1a43524772d991b5c64422189643cde61c3826 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2079547 Tested-by: Pirama Arumuga Nainar Reviewed-by: George Burgess --- compiler_wrapper/config.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 'compiler_wrapper/config.go') diff --git a/compiler_wrapper/config.go b/compiler_wrapper/config.go index 5f389684..70c27f06 100644 --- a/compiler_wrapper/config.go +++ b/compiler_wrapper/config.go @@ -14,6 +14,8 @@ type config struct { isAndroidWrapper bool // Whether to use ccache. useCCache bool + // Whether llvmNext wrapper. + useLlvmNext bool // Flags to add to gcc and clang. commonFlags []string // Flags to add to gcc only. @@ -83,6 +85,7 @@ func getConfig(configName string, useCCache bool, useLlvmNext bool, version stri return nil, newErrorwithSourceLocf("unknown config name: %s", configName) } cfg.useCCache = useCCache + cfg.useLlvmNext = useLlvmNext if useLlvmNext { cfg.clangFlags = append(cfg.clangFlags, llvmNextFlags...) } @@ -208,5 +211,5 @@ var androidConfig = &config{ gccFlags: []string{}, clangFlags: []string{}, clangPostFlags: []string{}, - newWarningsDir: "/tmp/fatal_clang_warnings", + newWarningsDir: "", } -- cgit v1.2.3