Re: [PATCH] Properly deprecate -fipa-cp-alignment

2017-02-25 Thread Gerald Pfeifer
On Tue, 21 Feb 2017, Martin Jambor wrote: > I see, I suppose we should be consistent, I will commit the following, > which I have included in a bootstrap and test run on x86_64-linux. Do you also plan to add a note to gcc-7/changes.html? I think that would make sense. Gerald

Re: [PATCH] Properly deprecate -fipa-cp-alignment

2017-02-21 Thread Martin Jambor
Hi, On Fri, Feb 17, 2017 at 10:34:21AM +0100, Jakub Jelinek wrote: > On Fri, Feb 17, 2017 at 10:31:16AM +0100, Martin Jambor wrote: > > @@ -8066,12 +8065,8 @@ This flag is enabled by default at @option{-O3}. > > > > @item -fipa-cp-alignment > > @opindex -fipa-cp-alignment > > -When enabled, th

Re: [PATCH] Properly deprecate -fipa-cp-alignment

2017-02-17 Thread Jakub Jelinek
On Fri, Feb 17, 2017 at 10:31:16AM +0100, Martin Jambor wrote: > @@ -8066,12 +8065,8 @@ This flag is enabled by default at @option{-O3}. > > @item -fipa-cp-alignment > @opindex -fipa-cp-alignment > -When enabled, this optimization propagates alignment of function > -parameters to support better

Re: [PATCH] Properly deprecate -fipa-cp-alignment

2017-02-17 Thread Martin Jambor
Hi, On Wed, Feb 08, 2017 at 01:49:17PM +0100, Jakub Jelinek wrote: > On Wed, Feb 08, 2017 at 01:41:24PM +0100, Martin Jambor wrote: > > 2017-02-08 Martin Jambor > > > > * common.opt (-finstrument-functions-exclude-file-list): Remove Var > > and Optimization, Document as deprecated and

Re: [PATCH] Properly deprecate -fipa-cp-alignment

2017-02-09 Thread Jan Hubicka
> > Or make it an alias for -fipa-bit-cp. I double programs are going on -fno-ipa-alignment for longer term - it would be very fragile because inlining and other transformations could do the same. I would just ignore it and mention that it no longer has effect in changes.html (in entry decsrib

Re: [PATCH] Properly deprecate -fipa-cp-alignment

2017-02-08 Thread Joseph Myers
On Wed, 8 Feb 2017, Jakub Jelinek wrote: > > fipa-cp-alignment > > -Common Report Var(flag_ipa_cp_alignment) Optimization > > -Perform alignment discovery and propagation to make Interprocedural > > constant propagation stronger. > > +Common Report > > +This switch is deprecated. Use -fipa-bit-

Re: [PATCH] Properly deprecate -fipa-cp-alignment

2017-02-08 Thread Richard Biener
On Wed, Feb 8, 2017 at 1:49 PM, Jakub Jelinek wrote: > On Wed, Feb 08, 2017 at 01:41:24PM +0100, Martin Jambor wrote: >> 2017-02-08 Martin Jambor >> >> * common.opt (-finstrument-functions-exclude-file-list): Remove Var >> and Optimization, Document as deprecated and superseded by >

Re: [PATCH] Properly deprecate -fipa-cp-alignment

2017-02-08 Thread Jakub Jelinek
On Wed, Feb 08, 2017 at 01:41:24PM +0100, Martin Jambor wrote: > 2017-02-08 Martin Jambor > > * common.opt (-finstrument-functions-exclude-file-list): Remove Var > and Optimization, Document as deprecated and superseded by > -fipa-bit-cp. > * doc/invoke.texi (Option Summ

[PATCH] Properly deprecate -fipa-cp-alignment

2017-02-08 Thread Martin Jambor
Hi, the patch below is my attempt at properly deprecating switch -fipa-cp-alignment. I have not found any clear example to follow but hopefully I have not missed anything. If so, please let me know. So far I have only tested this with a normal build and "make info" but I will give it a proper b