Re: [PATCH, DOC] Enhance documentation of -fipa-ra option.

2016-07-19 Thread Jeff Law
On 07/19/2016 08:26 AM, Martin Liška wrote: On 07/19/2016 03:46 PM, Alexander Monakov wrote: So I decided to also mention this alias. Thoughts? I'd like the new ipa-ra text to go in, but perhaps you should consider leaving out option aliases out of this patch, especially given that it's non-t

Re: [PATCH, DOC] Enhance documentation of -fipa-ra option.

2016-07-19 Thread Martin Liška
On 07/19/2016 03:46 PM, Alexander Monakov wrote: >> So I decided to also mention this alias. >> >> Thoughts? > > I'd like the new ipa-ra text to go in, but perhaps you should consider leaving > out option aliases out of this patch, especially given that it's non-trivial, > as > Andreas' comment h

Re: [PATCH, DOC] Enhance documentation of -fipa-ra option.

2016-07-19 Thread Alexander Monakov
> So I decided to also mention this alias. > > Thoughts? I'd like the new ipa-ra text to go in, but perhaps you should consider leaving out option aliases out of this patch, especially given that it's non-trivial, as Andreas' comment has shown. (I'd say it's rather confusing that -fprofile is qu

Re: [PATCH, DOC] Enhance documentation of -fipa-ra option.

2016-07-19 Thread Andreas Schwab
Martin Liška writes: > Thank you very much for the suggestions, I've basically followed all nits you > spotted. I also noticed that there is actually one more alias of -p and that's > -profile. That comes from >20 years old commit: -profile is not the same as -p, as the former also adds -lc_p.

Re: [PATCH, DOC] Enhance documentation of -fipa-ra option.

2016-07-19 Thread Martin Liška
On 07/13/2016 07:04 PM, Alexander Monakov wrote: >> --- a/gcc/doc/invoke.texi >> +++ b/gcc/doc/invoke.texi >> @@ -7260,7 +7260,9 @@ any called function. In that case it is not necessary >> to save and restore >> them around calls. This is only possible if called functions are part of >> same c

Re: [PATCH, DOC] Enhance documentation of -fipa-ra option.

2016-07-13 Thread Alexander Monakov
> --- a/gcc/doc/invoke.texi > +++ b/gcc/doc/invoke.texi > @@ -7260,7 +7260,9 @@ any called function. In that case it is not necessary > to save and restore > them around calls. This is only possible if called functions are part of > same compilation unit as current function and they are compil

[PATCH, DOC] Enhance documentation of -fipa-ra option.

2016-07-13 Thread Martin Liška
Hello. I accidentally learned that -fipa-ra option is disabled if: /* Do not use IPA optimizations for register allocation if profiler is active or port does not emit prologue and epilogue as RTL. */ if (profile_flag || !targetm.have_prologue () || !targetm.have_epilogue ()) flag_ipa_