[coreboot-gerrit] New patch to review for coreboot: 1c9431a util/xcompile/xcompile: Remove `-Wno-unused-but-set-variable` from `CFLAGS`

[Paul Menzel]

Paul Menzel ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3981

-gerrit

commit 1c9431aa0498113d1fb45a5d9bd266f139227e31
Author: Paul Menzel 
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 
---
 util/xcompile/xcompile | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/util/xcompile/xcompile b/util/xcompile/xcompile
index 5388889..9c54760 100644
--- a/util/xcompile/xcompile
+++ b/util/xcompile/xcompile
@@ -89,11 +89,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'