Re: [PATCH 2/3] function: Factor out make_*logue_seq

On 05/19/2016 01:16 AM, Jakub Jelinek wrote: On Wed, May 18, 2016 at 05:13:25PM -0500, Segher Boessenkool wrote: On Wed, May 18, 2016 at 01:35:16PM -0500, Segher Boessenkool wrote: On Wed, May 18, 2016 at 11:20:29AM -0700, H.J. Lu wrote: * function.c (make_split_prologue_seq,

Re: [PATCH 2/3] function: Factor out make_*logue_seq

On Thu, May 19, 2016 at 02:28:15AM -0500, Segher Boessenkool wrote: > The following fixes it, tested on x86_64-linux. Is it okay for trunk? > > (It only happens for targets that have a splitter gated by epilogue_done > for their eh_return pattern). > > > Segher > > > 2016-05-19 Segher

Re: [PATCH 2/3] function: Factor out make_*logue_seq

On Thu, May 19, 2016 at 09:16:26AM +0200, Jakub Jelinek wrote: > On Wed, May 18, 2016 at 05:13:25PM -0500, Segher Boessenkool wrote: > > On Wed, May 18, 2016 at 01:35:16PM -0500, Segher Boessenkool wrote: > > > On Wed, May 18, 2016 at 11:20:29AM -0700, H.J. Lu wrote: > > > > >> > *

Re: [PATCH 2/3] function: Factor out make_*logue_seq

On Wed, May 18, 2016 at 05:13:25PM -0500, Segher Boessenkool wrote: > On Wed, May 18, 2016 at 01:35:16PM -0500, Segher Boessenkool wrote: > > On Wed, May 18, 2016 at 11:20:29AM -0700, H.J. Lu wrote: > > > >> > * function.c (make_split_prologue_seq, make_prologue_seq, > > > >> >

Re: [PATCH 2/3] function: Factor out make_*logue_seq

On Wed, May 18, 2016 at 01:35:16PM -0500, Segher Boessenkool wrote: > On Wed, May 18, 2016 at 11:20:29AM -0700, H.J. Lu wrote: > > >> > * function.c (make_split_prologue_seq, make_prologue_seq, > > >> > make_epilogue_seq): New functions, factored out from... > > >> >

Re: [PATCH 2/3] function: Factor out make_*logue_seq

On Wed, May 18, 2016 at 11:20:29AM -0700, H.J. Lu wrote: > >> > * function.c (make_split_prologue_seq, make_prologue_seq, > >> > make_epilogue_seq): New functions, factored out from... > >> > (thread_prologue_and_epilogue_insns): Here. > >> > >> It breaks x86: > > > > Are

Re: [PATCH 2/3] function: Factor out make_*logue_seq

On Wed, May 18, 2016 at 11:20 AM, H.J. Lu wrote: > On Wed, May 18, 2016 at 11:11 AM, Segher Boessenkool > wrote: >> On Wed, May 18, 2016 at 10:17:32AM -0700, H.J. Lu wrote: >>> On Mon, May 16, 2016 at 6:09 PM, Segher Boessenkool >>>

Re: [PATCH 2/3] function: Factor out make_*logue_seq

On Wed, May 18, 2016 at 11:11 AM, Segher Boessenkool wrote: > On Wed, May 18, 2016 at 10:17:32AM -0700, H.J. Lu wrote: >> On Mon, May 16, 2016 at 6:09 PM, Segher Boessenkool >> wrote: >> > Make new functions make_split_prologue_seq,

Re: [PATCH 2/3] function: Factor out make_*logue_seq

On Wed, May 18, 2016 at 10:17:32AM -0700, H.J. Lu wrote: > On Mon, May 16, 2016 at 6:09 PM, Segher Boessenkool > wrote: > > Make new functions make_split_prologue_seq, make_prologue_seq, and > > make_epilogue_seq. > > > > Tested as in the previous patch; is this okay

Re: [PATCH 2/3] function: Factor out make_*logue_seq

On Mon, May 16, 2016 at 6:09 PM, Segher Boessenkool wrote: > Make new functions make_split_prologue_seq, make_prologue_seq, and > make_epilogue_seq. > > Tested as in the previous patch; is this okay for trunk? > > > Segher > > > 2016-05-16 Segher Boessenkool

Re: [PATCH 2/3] function: Factor out make_*logue_seq

On 05/16/2016 07:09 PM, Segher Boessenkool wrote: Make new functions make_split_prologue_seq, make_prologue_seq, and make_epilogue_seq. Tested as in the previous patch; is this okay for trunk? Segher 2016-05-16 Segher Boessenkool * function.c

[PATCH 2/3] function: Factor out make_*logue_seq

Make new functions make_split_prologue_seq, make_prologue_seq, and make_epilogue_seq. Tested as in the previous patch; is this okay for trunk? Segher 2016-05-16 Segher Boessenkool * function.c (make_split_prologue_seq, make_prologue_seq,