> This should be cfa_base_rtx instead of arg_pointer_rtx, or the condition > should test that cfa_base_rtx == arg_pointer_rtx. We don't have a dynamic > stack realignment target which defines FRAME_POINTER_CFA_OFFSET right now, > but what if one day such target is added or just dynamic realignment > support is added.
OK, I wondered about that in the original patch. Will adjust and re-test. -- Eric Botcazou