Re: [PATCH][AArch64] Simplify frame pointer logic

2018-05-22 Thread James Greenhalgh
On Tue, May 22, 2018 at 10:37:30AM -0500, Wilco Dijkstra wrote: > James Greenhalgh wrote: > > > +/* Determine whether a frame chain needs to be generated.  */ > > +static bool > > +aarch64_needs_frame_chain (void) > > +{ > > +  /* Force a frame chain for EH returns so the return address is at FP+8

Re: [PATCH][AArch64] Simplify frame pointer logic

2018-05-22 Thread Wilco Dijkstra
James Greenhalgh wrote: > +/* Determine whether a frame chain needs to be generated.  */ > +static bool > +aarch64_needs_frame_chain (void) > +{ > +  /* Force a frame chain for EH returns so the return address is at FP+8.  */ > +  if (frame_pointer_needed || crtl->calls_eh_return) > +    return tr

Re: [PATCH][AArch64] Simplify frame pointer logic

2018-05-22 Thread James Greenhalgh
On Tue, May 15, 2018 at 08:11:21AM -0500, Wilco Dijkstra wrote: > > ping > > > From: Wilco Dijkstra > Sent: 25 October 2017 16:29 > To: GCC Patches > Cc: nd > Subject: [PATCH][AArch64] Simplify frame pointer logic >   > > Simplify frame pointer logic based on review comments here > (https://gc

Re: [PATCH][AArch64] Simplify frame pointer logic

2018-05-15 Thread Wilco Dijkstra
ping From: Wilco Dijkstra Sent: 25 October 2017 16:29 To: GCC Patches Cc: nd Subject: [PATCH][AArch64] Simplify frame pointer logic   Simplify frame pointer logic based on review comments here (https://gcc.gnu.org/ml/gcc-patches/2017-10/msg01727.html). This patch incrementally adds to these f