From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 3959 invoked by alias); 30 Nov 2007 11:49:07 -0000 Received: (qmail 3950 invoked by uid 22791); 30 Nov 2007 11:49:06 -0000 X-Spam-Check-By: sourceware.org Received: from nz-out-0506.google.com (HELO nz-out-0506.google.com) (64.233.162.226) by sourceware.org (qpsmtpd/0.31) with ESMTP; Fri, 30 Nov 2007 11:48:54 +0000 Received: by nz-out-0506.google.com with SMTP id i11so2012899nzh for ; Fri, 30 Nov 2007 03:48:52 -0800 (PST) Received: by 10.142.132.2 with SMTP id f2mr1123365wfd.1196423331318; Fri, 30 Nov 2007 03:48:51 -0800 (PST) Received: by 10.143.44.8 with HTTP; Fri, 30 Nov 2007 03:48:51 -0800 (PST) Message-ID: <571f6b510711300348x2af444ddo71c0f989aab879a2@mail.gmail.com> Date: Fri, 30 Nov 2007 18:17:00 -0000 From: "Steven Bosscher" To: "Richard Guenther" Subject: Re: PR33713: remove -fforce-addr Cc: "Michael Meissner" , "Aldy Hernandez" , gcc-patches@gcc.gnu.org, bonzini@gnu.org, matz@suse.de, christophe.harle@amd.com In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <20071129175051.GA12296@redhat.com> <20071130000233.GA19742@mmeissner-gold.amd.com> Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org X-SW-Source: 2007-11/txt/msg01707.txt.bz2 On Nov 30, 2007 11:06 AM, Richard Guenther wrote: > > I also would prefer a deprecated switch get a warning, rather than silently > > accepting it. > > In general I agree here, but I don't have a strong opinion for > optimization switches. The problem with a warning is that people might be building with -Werror. If they do that and they also have the deprecated flag in their Makefile, their builds fail. But telling users that their build flags have no effect is IMHO a good thing. Proposal: We could add a "Deprecated" option property to common.opt, and use "inform()" to tell users they are using a deprecated option. If this sounds acceptable to y'all, I will craft a patch to implement it. (I couldn't find if this was proposed before. "inform()" is quite new, and I don't know if it was around already when it was first discussed what to do with removed options.) Gr. Steven