Re: [PATCH v3] add -fprolog-pad=N,M option

2016-12-20 Thread Maxim Kuvyrkov
> On Dec 19, 2016, at 6:04 PM, Bernd Schmidt wrote: > > I'll consider myself agnostic as to whether this is a feature we want or need, Hi Bernd, thanks for reviewing this! Regarding the usefulness of this feature, it has been discussed here (2 years ago): http://gcc.gcc.gnu.narkive.com/JfWUDn

Re: [PATCH v3] add -fprolog-pad=N,M option

2016-12-19 Thread Bernd Schmidt
I'll consider myself agnostic as to whether this is a feature we want or need, so I'll just comment on some style questions. There's a fair amount of coding style violations, I'll point some of them out but please read the documents we have linked on this page: https://gcc.gnu.org/contribute.

[PATCH v3] add -fprolog-pad=N,M option

2016-12-16 Thread Torsten Duwe
On Tue, Dec 06, 2016 at 05:02:36PM +0300, Maxim Kuvyrkov wrote: > > On Oct 5, 2016, at 12:45 AM, Maxim Kuvyrkov > > wrote: > > > > Ideally, I want to improve support for -fprolog-pad=N and > > __attribute__((prolog_pad(N))) to provide functionality to also output pad > > before the function la