From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 16494 invoked by alias); 26 Aug 2003 17:34:36 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 16469 invoked from network); 26 Aug 2003 17:34:35 -0000 Received: from unknown (HELO touchme.toronto.redhat.com) (207.219.125.105) by sources.redhat.com with SMTP; 26 Aug 2003 17:34:35 -0000 Received: from redhat.com (toocool.toronto.redhat.com [172.16.14.72]) by touchme.toronto.redhat.com (Postfix) with ESMTP id C33CC80034E; Tue, 26 Aug 2003 13:34:34 -0400 (EDT) Message-ID: <3F4B9A2A.6020901@redhat.com> Date: Tue, 26 Aug 2003 17:34:00 -0000 From: "J. Johnston" Organization: Red Hat Inc. User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4) Gecko/20030624 Netscape/7.1 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Kevin Buettner Cc: gdb-patches@sources.redhat.com Subject: Re: RFA: patch to display ia64 function pointers References: <3F2991AA.2050400@redhat.com> <1030825211237.ZM7510@localhost.localdomain> In-Reply-To: <1030825211237.ZM7510@localhost.localdomain> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2003-08/txt/msg00447.txt.bz2 Kevin Buettner wrote: > On Jul 31, 6:01pm, J. Johnston wrote: > > >>The following patch fixes the problem when displaying structs or function >>pointers that actually point to a function descriptor per the ia64 ABI. >>Currently, the function descriptor pointer is erroneously interpreted as >>pointing to a function and so gdb spits out some meaningless information. >> >>With the patch, you get to see the actual function that the pointer >>is referencing via the function descriptor. >> >>Ok to commit? >> >>2003-07-31 Jeff Johnston >> >> * ia64-tdep.c (ia64_convert_from_func_addr): New function. >> (ia64_gdbarch_init): Call set_gdbarch_convert_from_func_addr(). > > > Okay. > > Thanks, > > Kevin > > P.S. Please let me know if you're waiting on me to review any more of > your patches. > Patch checked in, thanks. No outstanding ia64 patches at the moment. There is a testsuite change to dump.exp to accomodate ia64 which I have submitted. I am also about to submit a change to prologue examination to handle glibc's sleep() function. -- Jeff J.