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

2017-02-17 Thread Torsten Duwe
On Wed, Feb 08, 2017 at 12:48:56PM +0100, Jakub Jelinek wrote: > > First of all, GCC is in stage4 and this isn't a fix for a regression, so it > is not acceptable at this point. Next stage1 will start in April or so. I had gotten the impression there were sort of branches in the SCM; this of cou

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

2017-02-17 Thread Torsten Duwe
On Wed, Feb 15, 2017 at 11:01:16AM +, Richard Earnshaw (lists) wrote: > On 13/01/17 12:19, Torsten Duwe wrote: > > > +++ b/gcc/doc/invoke.texi > > @@ -11341,6 +11341,27 @@ of the function name, it is considered to be a > > match. For C99 and C++ > > extended identifiers, the function name m

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

2017-02-15 Thread Richard Earnshaw (lists)
On 15/02/17 11:12, Marek Polacek wrote: > On Wed, Feb 15, 2017 at 11:01:16AM +, Richard Earnshaw (lists) wrote: >> On 13/01/17 12:19, Torsten Duwe wrote: >>> Changes since v4: hopefully addressed all of Sandra's requests >>> and suggestions concerning the documentation snippets, thanks >>> for

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

2017-02-15 Thread Marek Polacek
On Wed, Feb 15, 2017 at 11:01:16AM +, Richard Earnshaw (lists) wrote: > On 13/01/17 12:19, Torsten Duwe wrote: > > Changes since v4: hopefully addressed all of Sandra's requests > > and suggestions concerning the documentation snippets, thanks > > for the feedback. If it still isn't clear, fee

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

2017-02-15 Thread Richard Earnshaw (lists)
On 13/01/17 12:19, Torsten Duwe wrote: > Changes since v4: hopefully addressed all of Sandra's requests > and suggestions concerning the documentation snippets, thanks > for the feedback. If it still isn't clear, feel free to rephrase > -- I'm a programmer, not a technical writer. > Generally, I

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

2017-02-08 Thread Jakub Jelinek
On Wed, Feb 08, 2017 at 12:18:05PM +0100, Torsten Duwe wrote: > On Mon, Jan 23, 2017 at 05:25:45PM +0100, Bernd Schmidt wrote: > > There's still a a few details that need addressing, and some questions I > > have. Also Ccing Jakub to have another pair of eyes on the name of the > > section - I don'

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

2017-02-08 Thread Torsten Duwe
On Mon, Jan 23, 2017 at 05:25:45PM +0100, Bernd Schmidt wrote: > There's still a a few details that need addressing, and some questions I > have. Also Ccing Jakub to have another pair of eyes on the name of the > section - I don't know if we want some sort of .gnu.something name. So Jakub, Richard

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

2017-01-23 Thread Bernd Schmidt
There's still a a few details that need addressing, and some questions I have. Also Ccing Jakub to have another pair of eyes on the name of the section - I don't know if we want some sort of .gnu.something name. On 01/13/2017 01:19 PM, Torsten Duwe wrote: 2017-01-13 Torsten Duwe :

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

2017-01-23 Thread Torsten Duwe
And what's next now? Maxim mentioned in this thread you should sort of formally approve(?) a global feature, have you had a look at it yet? I should emphasize that this code can substitute the -fentry for arm64 for the purpose of kernel live patching that was discussed in the beginning, as well a

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

2017-01-15 Thread Sandra Loosemore
On 01/13/2017 05:19 AM, Torsten Duwe wrote: Changes since v4: hopefully addressed all of Sandra's requests and suggestions concerning the documentation snippets, thanks for the feedback. If it still isn't clear, feel free to rephrase -- I'm a programmer, not a technical writer. Thanks, this ma