Re: [PATCH 3/3] function: Restructure *logue insertion

2016-05-20 Thread Segher Boessenkool
On Fri, May 20, 2016 at 11:28:25AM +0200, Thomas Schwinge wrote: > > > > * function.c (make_epilogue_seq): Remove epilogue_end parameter. > > > > (thread_prologue_and_epilogue_insns): Remove bb_flags. Restructure > > > > code. Ignore sibcalls on EDGE_IGNORE edges. > > > > * shrink-wrap.c

Re: [PATCH 3/3] function: Restructure *logue insertion

2016-05-20 Thread Segher Boessenkool
On Fri, May 20, 2016 at 10:47:19AM -0400, Nathan Sidwell wrote: > On 05/20/16 09:21, Thomas Schwinge wrote: > >Hi! > > > >The nvptx maintainer Bernd, Nathan: can you take it from here, or should > >I continue to figure it out? > > What is the defect? I have a fix, testing now. Segher

Re: [PATCH 3/3] function: Restructure *logue insertion

2016-05-20 Thread Nathan Sidwell
On 05/20/16 09:21, Thomas Schwinge wrote: Hi! The nvptx maintainer Bernd, Nathan: can you take it from here, or should I continue to figure it out? What is the defect?

Re: [PATCH 3/3] function: Restructure *logue insertion

2016-05-20 Thread Thomas Schwinge
Hi! The nvptx maintainer Bernd, Nathan: can you take it from here, or should I continue to figure it out? On Fri, 20 May 2016 11:28:25 +0200, I wrote: > > > > * function.c (make_epilogue_seq): Remove epilogue_end parameter. > > > > (thread_prologue_and_epilogue_insns): Remove bb_flags.

Re: [PATCH 3/3] function: Restructure *logue insertion

2016-05-20 Thread Thomas Schwinge
Hi! > > > * function.c (make_epilogue_seq): Remove epilogue_end parameter. > > > (thread_prologue_and_epilogue_insns): Remove bb_flags. Restructure > > > code. Ignore sibcalls on EDGE_IGNORE edges. > > > * shrink-wrap.c (handle_simple_exit): New function. Set EDGE_IGNORE > > > on

Re: [PATCH 3/3] function: Restructure *logue insertion

2016-05-19 Thread Segher Boessenkool
On Thu, May 19, 2016 at 04:00:22PM -0600, Jeff Law wrote: > > * function.c (make_epilogue_seq): Remove epilogue_end parameter. > > (thread_prologue_and_epilogue_insns): Remove bb_flags. Restructure > > code. Ignore sibcalls on EDGE_IGNORE edges. > > * shrink-wrap.c

Re: [PATCH 3/3] function: Restructure *logue insertion

2016-05-19 Thread Jeff Law
On 05/16/2016 07:09 PM, Segher Boessenkool wrote: This patch restructures how the prologues/epilogues are inserted. Sibcalls that run without prologue are now handled in shrink-wrap.c; it communicates what is already handled by setting the EDGE_IGNORE flag. The try_shrink_wrapping function

Re: [PATCH 3/3] function: Restructure *logue insertion

2016-05-19 Thread Segher Boessenkool
On Tue, May 17, 2016 at 01:09:11AM +, Segher Boessenkool wrote: > This patch restructures how the prologues/epilogues are inserted. Sibcalls > that run without prologue are now handled in shrink-wrap.c; it communicates > what is already handled by setting the EDGE_IGNORE flag. The >

[PATCH 3/3] function: Restructure *logue insertion

2016-05-16 Thread Segher Boessenkool
This patch restructures how the prologues/epilogues are inserted. Sibcalls that run without prologue are now handled in shrink-wrap.c; it communicates what is already handled by setting the EDGE_IGNORE flag. The try_shrink_wrapping function then doesn't need to be passed the bb_flags anymore.