Hi Max,
> On Tue, Jun 18, 2019 at 4:53 PM Wilco Dijkstra <wilco.dijks...@arm.com> wrote:
> > > It would work if a frame pointer was initialized in the function test, but
> > > it wasn't:
> >
> > Right, because it unwinds, it needs a valid frame pointer since we no
> > longer store the stack pointer. So xtensa_frame_pointer_required
> > should do something like:
> >
> > if (cfun->machine->accesses_prev_frame || cfun->has_nonlocal_label)
> > return true;
>
> You're right, with this change things are back to normal.
Great, I've added this to the commit. Thanks for the proactive testing!
Wilco
