X-Recipient: archive-cygwin AT delorie DOT com X-Spam-Check-By: sourceware.org Date: Thu, 8 Oct 2009 14:57:42 -0400 From: Christopher Faylor To: cygwin AT cygwin DOT com Subject: Re: gdb, insight, and tcltk Message-ID: <20091008185742.GA1150@ednor.casa.cgf.cx> Reply-To: cygwin AT cygwin DOT com Mail-Followup-To: cygwin AT cygwin DOT com References: <4ACD0C50 DOT 2010902 AT cwilson DOT fastmail DOT fm> <4ACE3545 DOT 7070904 AT cwilson DOT fastmail DOT fm> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4ACE3545.7070904@cwilson.fastmail.fm> User-Agent: Mutt/1.5.20 (2009-06-14) Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: cygwin-owner AT cygwin DOT com Mail-Followup-To: cygwin AT cygwin DOT com Delivered-To: mailing list cygwin AT cygwin DOT com On Thu, Oct 08, 2009 at 02:53:57PM -0400, Charles Wilson wrote: >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? Use the insight mailing list, Luke. Keith Seitz is a friend and he does not hate Cygwin. cgf -- 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