Re: [PATCH 4/5 v7] Introduce common-debug.h

[Mike Frysinger <vapier@gentoo.org>]

[-- Attachment #1: Type: text/plain, Size: 2316 bytes --]

On Wed 20 Aug 2014 09:52:43 Gary Benson wrote:
> [Cc'ing Mike and Frank (SIM global maintainer and SIM common
> maintainer)]
> 
> Steve Ellcey wrote:
> > On Tue, 2014-08-19 at 10:12 +0100, Gary Benson wrote:
> > > > > 2014-08-13  Tom Tromey  <tromey@redhat.com>
> > > > > 
> > > > >             Gary Benson  <gbenson@redhat.com>
> > > > >         
> > > > >         * common/common-debug.h: New file.
> > > > >         * common/common-debug.c: Likewise.
> > > > >         * debug.c: Likewise.
> > > > >         * Makefile.in (SFILES): Add common/common-debug.c.
> > > > >         (HFILES_NO_SRCDIR): Add common/common-debug.h.
> > > > >         (COMMON_OBS): Add common-debug.o and debug.o.
> > > > >         (common-debug.o): New rule.
> > > > >         * common/common-defs.h: Include common-debug.h.
> > > > >         * common/agent.c (debug_agent_printf): New function.
> > > > >         (DEBUG_AGENT): Redefine.
> > > > >         * nat/i386-dregs.c (debug_printf): Undefine.
> > 
> > I just built gdb for mips (mips-linux-gnu and mips-mti-linux-gnu)
> > and it died with:
> > 
> > /local/home/sellcey/gcc/non_mti_linux/src/binutils-gdb/sim/mips/../common/
> > sim-trace.c:1326: multiple definition of `debug_printf'
> > common-debug.o:/local/home/sellcey/gcc/non_mti_linux/src/binutils-gdb/gdb
> > /common/common-debug.c:31: first defined here collect2: ld returned 1 exit
> > status
> > make[1]: *** [gdb] Error 1
> > 
> > I am assuming it has something to do with this patch.
> 
> Yeah.  This patch adds a "debug_printf" function to GDB, but it looks
> like SIM already had one.
> 
> I don't see any callers for SIM's debug_printf.  Is this dead code
> that can be removed, or is it provided as a convenience/library
> function?

it is not currently being used, but it really really should be in some targets

> Steve, you can probably work around this for now by adding:
> 
>   #define debug_printf gdb_debug_printf
> 
> at the top of GDB's common/common-debug.h.  If SIM's debug_printf
> cannot be removed then maybe we will have to add some hack like
> this to GDB permanently.

i'm fine with updating sim/common/sim-trace.h with that idea:
#define debug_printf sim_debug_printf

all the symbols coming out of the sim really should be namespaced
-mike

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 819 bytes --]