From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 127362 invoked by alias); 30 Aug 2017 10:45:42 -0000 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 Received: (qmail 127166 invoked by uid 89); 30 Aug 2017 10:45:41 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=BAYES_00,SPF_PASS autolearn=ham version=3.3.2 spammy=HContent-Transfer-Encoding:8bit X-HELO: mx1.suse.de Received: from mx2.suse.de (HELO mx1.suse.de) (195.135.220.15) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 30 Aug 2017 10:45:39 +0000 Received: from relay1.suse.de (charybdis-ext.suse.de [195.135.220.254]) by mx1.suse.de (Postfix) with ESMTP id C255AABA6; Wed, 30 Aug 2017 10:45:37 +0000 (UTC) Subject: Re: [RFC] [PATCH] Introduce configure flag --with-stage1-cflags. To: Richard Biener , Jeff Law Cc: Jakub Jelinek , GCC Patches References: <41109217-1bf5-b112-e783-8040196fd410@suse.cz> <20170526115155.GL8499@tucnak> From: =?UTF-8?Q?Martin_Li=c5=a1ka?= Message-ID: <8112344d-edff-2f5b-7e94-7e5b63ce858d@suse.cz> Date: Wed, 30 Aug 2017 12:19:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-IsSubscribed: yes X-SW-Source: 2017-08/txt/msg01713.txt.bz2 On 08/28/2017 02:24 PM, Richard Biener wrote: > On Fri, Aug 25, 2017 at 9:51 PM, Jeff Law wrote: >> On 07/31/2017 01:47 AM, Martin Liška wrote: >>> I would like to ping this. Input from other people will be appreciated ;) >> I think the thing to keep in mind here is that IIUC this only affects >> things when we've configured using the --with-stage1-cflags option. >> >> So questions about is -O1 more stable than -O2, should we restrict -O2 >> to newer compilers, etc are really more about the defaults we set. >> >> My understanding is the patch is just adding the capability and does not >> change the default. Assuming that's the case, then I'm comfortable >> acking the raw infrastructure. > > OTOH you can simply set STAGE1_CFLAGS so the value of this > as a configure option is somewhat questionable. Yes, STAGE1_CFLAGS is a working solution. I'm not planning to install the patch. Martin > >> jeff