On 07/25/2017 07:58 AM, Wilco Dijkstra wrote: > This patch makes some changes to the frame layout in order to simplify > stack probing. We want to use the save of LR as a probe in any non-leaf > function. With shrinkwrapping we may only save LR before a call, so it > is useful to define a fixed location in the callee-saves. So force LR at > the bottom of the callee-saves even with -fomit-frame-pointer. > > Also remove a rarely used frame layout that saves the callee-saves first > with -fomit-frame-pointer. > > OK for commit (and backport to GCC7)? > > ChangeLog: > 2017-07-25 Wilco Dijkstra <wdijk...@arm.com> > > * config/aarch64/aarch64.c (aarch64_layout_frame): > Ensure LR is always stored at the bottom of the callee-saves. > Remove frame option which saves callee-saves at top of frame. I'll let the appropriate aarch64 maintainers comment on correctness. But I wanted to give an explicit thanks for simplifying this so that we can rely on it.
Jeff