Re: RFA: RL78: Save the frame pointer if it is used.

2015-05-06 Thread Nicholas Clifton
Hi DJ, OK to apply ? Ok, but... Thanks - committed. - if (regno == FRAME_POINTER_REGNUM && frame_pointer_needed) + if (regno == FRAME_POINTER_REGNUM + && (frame_pointer_needed || df_regs_ever_live_p (regno))) Do we want regs_ever_live or regs_ever_written_to ? I seem to recall

Re: RFA: RL78: Save the frame pointer if it is used.

2015-05-05 Thread DJ Delorie
> OK to apply ? Ok, but... > - if (regno == FRAME_POINTER_REGNUM && frame_pointer_needed) > + if (regno == FRAME_POINTER_REGNUM > + && (frame_pointer_needed || df_regs_ever_live_p (regno))) Do we want regs_ever_live or regs_ever_written_to ? I seem to recall changing a port... mep per

RFA: RL78: Save the frame pointer if it is used.

2015-05-05 Thread Nick Clifton
Hi DJ, Below is a small patch to fix a small problem with the need_to_save() function in the RL78 backend. It was only marking the frame pointer register as needing a save if the frame pointer was in use. But since the frame pointer is call-saved it needs to be saved any time it is use