[RFC/commit] Getting rid of tm-rs6000.h (PROCESS_LINENUMBER_HOOK)

Joel Brobecker brobecker@adacore.com
Thu Sep 20 06:37:00 GMT 2007


Hi Mark,

> > The AIX port is the only port that defines this macro, and it's only
> > used in one place: buildsym.c:end_symtab(). I think the most appropriate
> > way to define this notion is as a gdbarch method, set only in the case
> > of AIX. This is what this patch does.
> 
> Hmm, if this is something that is purely for a specific object/debug format,
> I think the gdbarch vector is the wrong place to put it.

Since the gdbarch vector is dependent on the ABI, we can set the method
when identifying XCOFF objects. That seemed the simplest approach at
the time.  But I agree that this is borderline. What do you think about
adding a new method inside struct sym_fns: 

        void (*sym_read_linetable) (void)

It would be null for all object formats except XCOFF.

I'll experiment with that... Thanks for your feedback!
-- 
Joel



More information about the Gdb-patches mailing list