This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH]: Updated patch to output dwarf2 namespace debug info


On Mon, 1 Dec 2003 11:58:26 -0600, Benjamin Kosnik <bkoz@redhat.com> said:

> Attached is a patch that applies without errors to today's sources. I
> have verified that it bootstraps and adds no new regressions. 

Thanks to everybody who's gotten this applied so quickly - I really
appreciate it.

> Where are the patches to gdb that make use of this info? I'd like to
> try them out.

Do:

cvs -d :pserver:anoncvs@sources.redhat.com:/cvs/src co -r carlton_dictionary-branch gdb+dejagnu

To run tests, go to the gdb directory and do 'make check'.  The test
results shouldn't be any worse after this patch has been applied to
GCC than they were before the patch was applied; one test, in
rtti.exp, should turn from a FAIL (or KFAIL, I can't remember) to
PASS.

One question:
  
> + /* Force out any required namespaces to be able to output DECL,
> +    and return the new context_die for it, if it's changed.  */
> + static dw_die_ref
> + setup_namespace_context (tree decl, dw_die_ref context_die)
> + {
> +    /* Force out the namespace. */
> +   if (!DECL_ABSTRACT (decl) && DECL_CONTEXT (decl) && TREE_CODE (DECL_CONTEXT (decl)) == NAMESPACE_DECL)

The version of this patch that I've been working with doesn't have the
"!DECL_ABSTRACT(decl)" part of this.  What's its purpose?  (I don't
know much about GCC's internals.)

David Carlton
carlton@kealia.com


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]