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