From c3bb0cbac10890324c5004fd6d7819102a2e6995 Mon Sep 17 00:00:00 2001 From: Pirama Arumuga Nainar Date: Wed, 26 Feb 2020 17:59:58 -0800 Subject: When double-build-with-Wno-error, add -Wno-error=foo for -Werror=foo This will gracefully handle cases where a warning enforced with -Werror got stricter in upstream. This change also needs updates to golden testdata. No additional tests added since this is covered by existing tests. Originally reviewed in https://android-review.googlesource.com/c/1244796. BUG=b:149836702 TEST=go test Change-Id: I94624deb258b15e131387377b99bfb1747dca1a1 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2079548 Tested-by: Pirama Arumuga Nainar Reviewed-by: George Burgess --- .../testdata/cros_clang_host_golden/force_disable_werror.json | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'compiler_wrapper/testdata/cros_clang_host_golden') diff --git a/compiler_wrapper/testdata/cros_clang_host_golden/force_disable_werror.json b/compiler_wrapper/testdata/cros_clang_host_golden/force_disable_werror.json index 248fcb4b..2393e71a 100644 --- a/compiler_wrapper/testdata/cros_clang_host_golden/force_disable_werror.json +++ b/compiler_wrapper/testdata/cros_clang_host_golden/force_disable_werror.json @@ -94,7 +94,8 @@ "-Wno-unknown-warning-option", "main.cc", "-Wno-implicit-int-float-conversion", - "-Wno-error" + "-Wno-error", + "-Wno-error=poison-system-directories" ] } } @@ -159,7 +160,8 @@ "-Wno-unknown-warning-option", "main.cc", "-Wno-implicit-int-float-conversion", - "-Wno-error" + "-Wno-error", + "-Wno-error=poison-system-directories" ] }, "stdout": "somemessage", -- cgit v1.2.3