[rfc]: Replace current_gdbarch in sentinel-frame.c

Markus Deuling deuling@de.ibm.com
Mon Feb 18 15:28:00 GMT 2008


Ulrich Weigand schrieb:
> Thiago Jung Bauermann wrote:
>> On Fri, 2008-02-15 at 18:43 +0100, Ulrich Weigand wrote:
>>> This is not OK -- as the sentinel frame is always the innermost
>>> frame, next_frame will always be NULL here.
>> Actually, what I observe here when running GDB is that next_frame is the
>> sentinel frame, probably because sentinel frame's next field points back
>> to itself.
> 
> Ah, right, I forgot about that.  Still, it seems best to try to get 
> rid of this hack completely ...
> 

I tested *addrp = 0 on x86 and saw no regression. I changed the patch to remove
register_offset_hack completely.

Ok ?


ChangeLog:

	* sentinel-frame.c (sentinel_frame_prev_register): Do not call
	register_offset_hack anymore.

	* regcache.{c,h} (register_offset_hack): Remove.


Regards,
Markus

-- 
  Markus Deuling
  GNU Toolchain for Linux on Cell BE
  deuling@de.ibm.com

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: diff-sentinel
URL: <http://sourceware.org/pipermail/gdb-patches/attachments/20080218/3a147e20/attachment.ksh>


More information about the Gdb-patches mailing list