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
     

Reply via email to