This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix debug info with crtl->stack_realign_tried (PR debug/40596)
- From: Jakub Jelinek <jakub at redhat dot com>
- To: "H.J. Lu" <hjl dot tools at gmail dot com>
- Cc: "Guo, Xuepeng" <xuepeng dot guo at intel dot com>, Richard Henderson <rth at redhat dot com>, Jason Merrill <jason at redhat dot com>, gcc-patches at gcc dot gnu dot org
- Date: Thu, 2 Jul 2009 15:08:32 +0200
- Subject: Re: [PATCH] Fix debug info with crtl->stack_realign_tried (PR debug/40596)
- References: <20090702112941.GO4462@tyan-ft48-01.lab.bos.redhat.com> <6dc9ffc80907020601x6d8f5178sb27617152ad9ab5@mail.gmail.com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
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