This is the mail archive of the
cygwin
mailing list for the Cygwin project.
Re: gdb, insight, and tcltk
- From: Charles Wilson <cygwin at cwilson dot fastmail dot fm>
- To: cygwin at cygwin dot com
- Date: Thu, 08 Oct 2009 14:53:57 -0400
- Subject: Re: gdb, insight, and tcltk
- References: <4ACD0C50.2010902@cwilson.fastmail.fm>
The existing cygwin gdb/insight 6.8-2 package has the following change:
2007-11-30 Pedro Alves <...>
* i386-tdep.c (struct i386_frame_cache): Rename saved_sp to
prev_frame_sp. Add saved_sp_regnum field.
(i386_alloc_frame_cache): Update.
(i386_analyze_stack_align): Record which register holds %esp in
saved_sp_regnum.
(i386_analyze_register_saves): Move higher on the file.
(i386_analyze_frame_setup): Account for register saves before stack
adjustment.
(i386_frame_cache): If possible, prefer reading the register that holds
the previous stack pointer from the stack .
(i386_frame_prev_register): Update.
(i386_frame_args_address): New.
(i386_frame_base): Set i386_frame_args_address as args method.
(patch vs. "regular" gdb-6.8 attached). However, I *think* that given this:
http://sourceware.org/ml/gdb-cvs/2008-08/msg00053.html
Discussion here:
http://sourceware.org/ml/gdb-patches/2008-07/msg00575.html
http://sourceware.org/ml/gdb-patches/2008-08/msg00141.html
that Pedro's change is now unnecessary (it certainly needs a lot of TLC
to apply to 7.0, in any case). Also: gdb/insight built without this
change seem to work ok.
Any thoughts?
--
Chuck
--
Problem reports: http://cygwin.com/problems.html
FAQ: http://cygwin.com/faq/
Documentation: http://cygwin.com/docs.html
Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple