Re: [RFC] [PATCH] Introduce configure flag --with-stage1-cflags.

2017-08-30 Thread Martin Liška
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 >>

Re: [RFC] [PATCH] Introduce configure flag --with-stage1-cflags.

2017-08-28 Thread Richard Biener
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

Re: [RFC] [PATCH] Introduce configure flag --with-stage1-cflags.

2017-08-25 Thread Jeff Law
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 th

Re: [RFC] [PATCH] Introduce configure flag --with-stage1-cflags.

2017-07-31 Thread Martin Liška
I would like to ping this. Input from other people will be appreciated ;) Thanks, Martin On 06/19/2017 02:30 PM, Richard Biener wrote: > On Mon, Jun 19, 2017 at 12:51 PM, Martin Liška wrote: >> PING^1 >> >> Richi are you fine with the suggested change? I basically followed your >> advises :) >

Re: [RFC] [PATCH] Introduce configure flag --with-stage1-cflags.

2017-06-19 Thread Richard Biener
On Mon, Jun 19, 2017 at 12:51 PM, Martin Liška wrote: > PING^1 > > Richi are you fine with the suggested change? I basically followed your > advises :) Well, I am but as Eric disagrees I think we need input from other people on this. I'm comfortably setting STAGE1_CFLAGS here. Richard. > Marti

Re: [RFC] [PATCH] Introduce configure flag --with-stage1-cflags.

2017-06-19 Thread Martin Liška
PING^1 Richi are you fine with the suggested change? I basically followed your advises :) Martin On 05/26/2017 03:00 PM, Martin Liška wrote: > On 05/26/2017 01:55 PM, Richard Biener wrote: >> On Fri, May 26, 2017 at 1:51 PM, Jakub Jelinek wrote: >>> On Fri, May 26, 2017 at 01:46:47PM +0200, Ri

Re: [RFC] [PATCH] Introduce configure flag --with-stage1-cflags.

2017-05-30 Thread Eric Botcazou
> That's why we decided to enable -O2 just with GCC 4.8+. IMO it's too dangerous on non-x86 platforms and -O1 would be much safer. -- Eric Botcazou

Re: [RFC] [PATCH] Introduce configure flag --with-stage1-cflags.

2017-05-30 Thread Martin Liška
On 05/30/2017 11:43 AM, Eric Botcazou wrote: >> Are you sure? In my experience -O2 gets the most test coverage during >> lifetime of a particular release. > > Yes, some older GCC releases have aliasing issues that are exposed at -O2 > only > because of -fstrict-aliasing and -fschedule-insns. >

Re: [RFC] [PATCH] Introduce configure flag --with-stage1-cflags.

2017-05-30 Thread Eric Botcazou
> Are you sure? In my experience -O2 gets the most test coverage during > lifetime of a particular release. Yes, some older GCC releases have aliasing issues that are exposed at -O2 only because of -fstrict-aliasing and -fschedule-insns. -- Eric Botcazou

Re: [RFC] [PATCH] Introduce configure flag --with-stage1-cflags.

2017-05-30 Thread Richard Biener
On Mon, May 29, 2017 at 1:13 PM, Eric Botcazou wrote: >> After a discussion with Richi, using adding "-O2" to STAGE1 cflags with a >> recent enough compiler can significantly speed up bootstrap. Thus I'm >> suggesting to introduce --with-stage1-cflags where one can provide such >> options. > > -O1

Re: [RFC] [PATCH] Introduce configure flag --with-stage1-cflags.

2017-05-29 Thread Eric Botcazou
> After a discussion with Richi, using adding "-O2" to STAGE1 cflags with a > recent enough compiler can significantly speed up bootstrap. Thus I'm > suggesting to introduce --with-stage1-cflags where one can provide such > options. -O1 is sufficient in my experience and far less risky than -O2 in

Re: [RFC] [PATCH] Introduce configure flag --with-stage1-cflags.

2017-05-26 Thread Martin Liška
On 05/26/2017 03:00 PM, Martin Liška wrote: > Ok, sending new patch that does that on experimental branches for ${CC} > --version > being a GCC newer than 4.9. s/4.9/4.8/

Re: [RFC] [PATCH] Introduce configure flag --with-stage1-cflags.

2017-05-26 Thread Martin Liška
On 05/26/2017 01:55 PM, Richard Biener wrote: > On Fri, May 26, 2017 at 1:51 PM, Jakub Jelinek wrote: >> On Fri, May 26, 2017 at 01:46:47PM +0200, Richard Biener wrote: >>> On Thu, May 25, 2017 at 11:23 AM, Martin Liška wrote: Hello. After a discussion with Richi, using adding "-O2

Re: [RFC] [PATCH] Introduce configure flag --with-stage1-cflags.

2017-05-26 Thread Richard Biener
On Fri, May 26, 2017 at 1:51 PM, Jakub Jelinek wrote: > On Fri, May 26, 2017 at 01:46:47PM +0200, Richard Biener wrote: >> On Thu, May 25, 2017 at 11:23 AM, Martin Liška wrote: >> > Hello. >> > >> > After a discussion with Richi, using adding "-O2" to STAGE1 cflags with a >> > recent >> > enough

Re: [RFC] [PATCH] Introduce configure flag --with-stage1-cflags.

2017-05-26 Thread Jakub Jelinek
On Fri, May 26, 2017 at 01:46:47PM +0200, Richard Biener wrote: > On Thu, May 25, 2017 at 11:23 AM, Martin Liška wrote: > > Hello. > > > > After a discussion with Richi, using adding "-O2" to STAGE1 cflags with a > > recent > > enough compiler can significantly speed up bootstrap. Thus I'm sugges

Re: [RFC] [PATCH] Introduce configure flag --with-stage1-cflags.

2017-05-26 Thread Richard Biener
On Thu, May 25, 2017 at 11:23 AM, Martin Liška wrote: > Hello. > > After a discussion with Richi, using adding "-O2" to STAGE1 cflags with a > recent > enough compiler can significantly speed up bootstrap. Thus I'm suggesting to > introduce --with-stage1-cflags where one can provide such options.

[RFC] [PATCH] Introduce configure flag --with-stage1-cflags.

2017-05-25 Thread Martin Liška
Hello. After a discussion with Richi, using adding "-O2" to STAGE1 cflags with a recent enough compiler can significantly speed up bootstrap. Thus I'm suggesting to introduce --with-stage1-cflags where one can provide such options. Apart from that, maybe it would be handy to automatically enable