[coreboot-gerrit] Patch set updated for coreboot: 2c73f6e util/xcompile/xcompile: Remove `-Wno-unused-but-set-variable` from `CFLAGS`
Paul Menzel (paulepanter@users.sourceforge.net)
gerrit at coreboot.org
Tue Nov 18 12:12:53 CET 2014
Paul Menzel (paulepanter at users.sourceforge.net) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3981
-gerrit
commit 2c73f6ea5ae25b3dc64efee29aaafe10604b71b6
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date: Sun Oct 20 00:11:34 2013 +0200
util/xcompile/xcompile: Remove `-Wno-unused-but-set-variable` from `CFLAGS`
Do not disable warnings about unused but set variables to further
improve the code quality.
Change-Id: I25fa29ac42c9d09596d03f11fb01f31635a62a11
Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
---
util/xcompile/xcompile | 5 -----
1 file changed, 5 deletions(-)
diff --git a/util/xcompile/xcompile b/util/xcompile/xcompile
index 7bb6e4c..588a8eb 100755
--- a/util/xcompile/xcompile
+++ b/util/xcompile/xcompile
@@ -120,11 +120,6 @@ testas() {
detect_special_flags() {
local architecture="$1"
- # GCC 4.6 is much more picky about unused variables.
- # Turn off it's warnings for now:
- testcc "$CC" "$CFLAGS -Wno-unused-but-set-variable " &&
- CFLAGS="$CFLAGS -Wno-unused-but-set-variable "
-
# Use bfd linker instead of gold if available:
testcc "$CC" "$CFLAGS -fuse-ld=bfd" &&
CFLAGS="$CFLAGS -fuse-ld=bfd" && LINKER_SUFFIX='.bfd'
More information about the coreboot-gerrit
mailing list