Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754317Ab1DUUqg (ORCPT ); Thu, 21 Apr 2011 16:46:36 -0400 Received: from mx1.redhat.com ([209.132.183.28]:40484 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752955Ab1DUUqf (ORCPT ); Thu, 21 Apr 2011 16:46:35 -0400 Date: Thu, 21 Apr 2011 16:45:49 -0400 From: Dave Jones To: David Daney Cc: Sam Ravnborg , Joe Perches , Linus Torvalds , Linux Kernel , Andrew Morton , BorislavPetkov Subject: Re: annoying new gcc 4.6.0 warnings. Message-ID: <20110421204549.GA20508@redhat.com> Mail-Followup-To: Dave Jones , David Daney , Sam Ravnborg , Joe Perches , Linus Torvalds , Linux Kernel , Andrew Morton , BorislavPetkov References: <20110421192117.GA14249@redhat.com> <4DB087EC.1010205@caviumnetworks.com> <1303415086.24766.84.camel@Joe-Laptop> <20110421195801.GA15126@redhat.com> <20110421200836.GA31709@merkur.ravnborg.org> <20110421202713.GA19425@redhat.com> <4DB09590.1040709@caviumnetworks.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4DB09590.1040709@caviumnetworks.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1642 Lines: 49 On Thu, Apr 21, 2011 at 01:37:36PM -0700, David Daney wrote: > > +KBUILD_EXTRA_WARNINGS += -Wunused-but-set-variable > > Well I didn't test it, but presumably if you need the $(call cc-option, > -Wno-unused-but-set-variable), you would need similar here. Third time's the charm ? Dave -- Disable the new -Wunused-but-set-variable that was added in gcc 4.6.0 It produces more false positives than useful warnings. This can still be enabled using W=1 Signed-off-by: Dave Jones diff --git a/Makefile b/Makefile index b967b96..29e16f2 100644 --- a/Makefile +++ b/Makefile @@ -559,6 +559,8 @@ ifndef CONFIG_CC_STACKPROTECTOR KBUILD_CFLAGS += $(call cc-option, -fno-stack-protector) endif +KBUILD_CFLAGS += $(call cc-option, -Wno-unused-but-set-variable) + ifdef CONFIG_FRAME_POINTER KBUILD_CFLAGS += -fno-omit-frame-pointer -fno-optimize-sibling-calls else diff --git a/scripts/Makefile.build b/scripts/Makefile.build index d5f925a..30627ab 100644 --- a/scripts/Makefile.build +++ b/scripts/Makefile.build @@ -79,6 +79,7 @@ KBUILD_EXTRA_WARNINGS += -Wpointer-arith KBUILD_EXTRA_WARNINGS += -Wredundant-decls KBUILD_EXTRA_WARNINGS += -Wshadow KBUILD_EXTRA_WARNINGS += -Wswitch-default +KBUILD_EXTRA_WARNINGS += $(call cc-option, -Wunused-but-set-variable) KBUILD_EXTRA_WARNINGS += $(call cc-option, -Wvla,) KBUILD_CFLAGS += $(KBUILD_EXTRA_WARNINGS) endif -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/