This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB 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: [RFC] add offset support to DWARF2_FRAME_REG_CFA

From: Daniel Jacobowitz <drow at false dot org>
To: Mark Kettenis <mark dot kettenis at xs4all dot nl>
Cc: christophe dot lyon at st dot com, gdb-patches at sources dot redhat dot com
Date: Fri, 25 Nov 2005 13:02:11 -0500
Subject: Re: [RFC] add offset support to DWARF2_FRAME_REG_CFA
References: <4382E5B0.3030702@st.com> <200511221859.jAMIxvQv003229@elgar.sibelius.xs4all.nl> <43842C4F.8070405@st.com> <200511242237.jAOMbpxi014472@elgar.sibelius.xs4all.nl>

On Thu, Nov 24, 2005 at 11:37:51PM +0100, Mark Kettenis wrote:
> Thanks!  You seemed to have forgotten about my suggestion to introduce
> DWARF2_FRAME_REG_CFA_OFFSET.  The problem is that as-is, your patch
> seems to break sparc64 and cris.  By introducing
> DWARF2_FRAME_CFA_OFFSET all existing code will be safe.

You've already approved the patch with this change, so that's fine, but
I don't follow your reasoning; why not just fix sparc64 and cris?

-- 
Daniel Jacobowitz
CodeSourcery, LLC

References:
- [RFC] add offset support to DWARF2_FRAME_REG_CFA
  - From: Christophe LYON
- Re: [RFC] add offset support to DWARF2_FRAME_REG_CFA
  - From: Mark Kettenis
- Re: [RFC] add offset support to DWARF2_FRAME_REG_CFA
  - From: Christophe LYON
- Re: [RFC] add offset support to DWARF2_FRAME_REG_CFA
  - From: Mark Kettenis

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