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
changing a port... mep perhaps... to only save registers that are
changed, not registers that are used but read-only.

You are right, we would only need to save the FP if it is written to, but I could not find a regs_ever_written_to, or something similar. At least the current version of the patch will still work though, even if there is a needless save/restore of the FP.

Cheers
  Nick


Reply via email to