From 534c5e113397a61f25c360467ae5956bed04db25 Mon Sep 17 00:00:00 2001 From: Paul Menzel Date: Sun, 20 Oct 2013 00:11:34 +0200 Subject: [PATCH] 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 Reviewed-on: https://review.coreboot.org/3981 Reviewed-by: Stefan Reinauer Tested-by: build bot (Jenkins) --- util/xcompile/xcompile | 5 ----- 1 file changed, 5 deletions(-) diff --git a/util/xcompile/xcompile b/util/xcompile/xcompile index 2b9b8189a9..e1fcb31cc0 100755 --- a/util/xcompile/xcompile +++ b/util/xcompile/xcompile @@ -171,11 +171,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 "$GCC" "$CFLAGS_GCC -Wno-unused-but-set-variable " && - CFLAGS_GCC="$CFLAGS_GCC -Wno-unused-but-set-variable " - # Check for an operational -m32/-m64 testcc "$GCC" "$CFLAGS_GCC -m$TWIDTH " && CFLAGS_GCC="$CFLAGS_GCC -m$TWIDTH "