This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Fix debug info with crtl->stack_realign_tried (PR debug/40596)


On Thu, Jul 02, 2009 at 06:01:25AM -0700, H.J. Lu wrote:
> I think the 2nd and 3rd patches should go together.

That's not going to work.  The second patch doesn't work without the first,
because without the first cfa variable at the end just describes CFA state
after last instruction (when not using epilogue unwind or when epilogue is
followed by further code, this is CFA state in the middle of the function,
but otherwise it is CFA state after the epilogue).

I think it is a bad idea to touch cfa after function body has been emitted
to assembly, so my preference is 3rd patch alone.

	Jakub


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]