From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 16063 invoked by alias); 5 Jul 2011 15:24:42 -0000 Received: (qmail 16053 invoked by uid 22791); 5 Jul 2011 15:24:41 -0000 X-SWARE-Spam-Status: No, hits=-0.6 required=5.0 tests=AWL,BAYES_50,T_RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from sibelius.xs4all.nl (HELO glazunov.sibelius.xs4all.nl) (83.163.83.176) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 05 Jul 2011 15:24:14 +0000 Received: from glazunov.sibelius.xs4all.nl (kettenis@localhost [127.0.0.1]) by glazunov.sibelius.xs4all.nl (8.14.3/8.14.3) with ESMTP id p65FOA7F005718; Tue, 5 Jul 2011 17:24:10 +0200 (CEST) Received: (from kettenis@localhost) by glazunov.sibelius.xs4all.nl (8.14.3/8.14.3/Submit) id p65FO8pL010651; Tue, 5 Jul 2011 17:24:08 +0200 (CEST) Date: Tue, 05 Jul 2011 15:34:00 -0000 Message-Id: <201107051524.p65FO8pL010651@glazunov.sibelius.xs4all.nl> From: Mark Kettenis To: sanjoy@playingwithpointers.com CC: gdb-patches@sourceware.org, sanjoy@playingwithpointers.com In-reply-to: <1309847489-13760-4-git-send-email-sanjoy@playingwithpointers.com> (message from Sanjoy Das on Tue, 5 Jul 2011 12:01:29 +0530) Subject: Re: [PATCH 4/4] Activate JIT code unwinder for i386. References: <1309847489-13760-1-git-send-email-sanjoy@playingwithpointers.com> <1309847489-13760-4-git-send-email-sanjoy@playingwithpointers.com> Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2011-07/txt/msg00152.txt.bz2 > From: Sanjoy Das > Date: Tue, 5 Jul 2011 12:01:29 +0530 > > Registers the (pseudo) unwinder for JIT code for i386. This can > probably be expanded to other architectures after some testing and > review. What's your motivation for putting the JIT unwinder at that spot in the list of unwinders? I'd put it behind the DWARF CFI unwinders since those are the ones that most people will hit. > gdb/ChangeLog: > * i386-tdep.c: Activate JIT unwinder for i386. > --- > gdb/ChangeLog | 4 ++++ > gdb/i386-tdep.c | 4 ++++ > 2 files changed, 8 insertions(+), 0 deletions(-) > > diff --git a/gdb/ChangeLog b/gdb/ChangeLog > index 306a0a5..1f0bd19 100644 > --- a/gdb/ChangeLog > +++ b/gdb/ChangeLog > @@ -1,5 +1,9 @@ > 2011-07-04 Sanjoy Das > > + * i386-tdep.c: Activate JIT unwinder for i386. > + > +2011-07-04 Sanjoy Das > + > * jit.c: Add JIT frame unwinding code. > * jit-reader.h: Added jit_prepend_unwinder. > > diff --git a/gdb/i386-tdep.c b/gdb/i386-tdep.c > index 366d0fa..66626ab 100644 > --- a/gdb/i386-tdep.c > +++ b/gdb/i386-tdep.c > @@ -30,6 +30,7 @@ > #include "frame-base.h" > #include "frame-unwind.h" > #include "inferior.h" > +#include "jit.h" > #include "gdbcmd.h" > #include "gdbcore.h" > #include "gdbtypes.h" > @@ -7330,6 +7331,9 @@ i386_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches) > CFI info will be used if it is available. */ > dwarf2_append_unwinders (gdbarch); > > + /* JIT reader pseudo-unwinder. */ > + jit_prepend_unwinder (gdbarch); > + > frame_base_set_default (gdbarch, &i386_frame_base); > > /* Pseudo registers may be changed by amd64_init_abi. */ > -- > 1.7.5.3 > >