Re: [PATCH] rs6000: Make deallocation of a large frame work (PR77687)

2016-11-28 Thread Segher Boessenkool
On Tue, Nov 15, 2016 at 09:06:23AM -0500, David Edelsohn wrote: > > * config/rs6000/rs6000.c (rs6000_emit_stack_reset): Emit the > > stack_restore_tie insn instead of stack_tie, for the SVR4 and > > SPE ABIs. > > * config/rs6000/rs6000.md (stack_restore_tie): New

Re: [PATCH] rs6000: Make deallocation of a large frame work (PR77687)

2016-11-15 Thread David Edelsohn
On Tue, Nov 15, 2016 at 6:48 AM, Segher Boessenkool wrote: > If we use ABI_V4 and we have a big stack frame, we end the epilogue > with a "mr 1,11" (or similar) instruction. This instruction however > has no dependencies on the earlier restores from stack (done via

[PATCH] rs6000: Make deallocation of a large frame work (PR77687)

2016-11-15 Thread Segher Boessenkool
If we use ABI_V4 and we have a big stack frame, we end the epilogue with a "mr 1,11" (or similar) instruction. This instruction however has no dependencies on the earlier restores from stack (done via r11), so sched2 can end up reordering the insns, which is bad because we have no red zone so