[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