Re: Patching gdb 5.0 for XFree86 module support

[Kevin Buettner <kevinb at cygnus dot com>]

On Sep 23,  1:42pm, Daniel Jacobowitz wrote:

> > --- gdb/dbxread.c.xfree86-modules	Sat Jul  7 13:19:50 2001
> > +++ gdb/dbxread.c	Sun Sep 23 07:53:12 2001
> > @@ -2272,7 +2272,7 @@
> >  	    case 'F':
> >  	      function_stab_type = type;
> >  
> > -#ifdef SOFUN_ADDRESS_MAYBE_MISSING
> > +#if defined(SOFUN_ADDRESS_MAYBE_MISSING) || defined(XFREE_MODULE_SUPPORT)
> >  	      /* Deal with the SunPRO 3.0 compiler which omits the address
> >  	         from N_FUN symbols.  */
> >  	      if (type == N_FUN
> 
> I'm not sure that's going to fly.  SOFUN_ADDRESS_MAYBE_MISSING
> introduces its own problems; there's a good discussion in the list
> archives but I can't find it at the moment.  Of course, I can see why
> you'd need that segment.

I agree with Daniel.  I think it's going to be difficult to get the
symtab maintainers to agree to this change.

I'm curious about why it was needed in the first place though.
SOFUN_ADDRESS_MAYBE_MISSING is alread defined for Linux.  Is it
needed for some other (non-Linux) platform?

Hmm...  it just occurred to me that SOFUN_ADDRESS_MAYBE_MISSING needs
to be multiarched.  Once that's done, your problem is solved since
this could be turned on or off at will.

Kevin