Re: [rfa+5.3] bug in my earlier DW_TAG_namespace patch

[Andrew Cagney <ac131313@redhat.com>]

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

Hello,

David recommended this for:

#######          #####
#               #     #
#                     #
######           #####
       #   ###         #
#     #   ###   #     #
  #####    ###    ####

any comments?

Andrew

[-- Attachment #2: mailbox-message://ac131313@movemail/fsf/gdb/patches#10783294 --]
[-- Type: message/rfc822, Size: 4395 bytes --]

From: David Carlton <carlton@math.stanford.edu>
To: gdb-patches@sources.redhat.com
Cc: Jim Blandy <jimb@redhat.com>, Elena Zannoni <ezannoni@redhat.com>
Subject: [rfa+5.3] bug in my earlier DW_TAG_namespace patch
Date: 21 Nov 2002 16:26:22 -0800
Message-ID: <ro1fztuh2kx.fsf@jackfruit.Stanford.EDU>

I just noticed that my earlier patch that allowed GDB to accept
DW_TAG_namespace entries had a bug in it.  The function
scan_partial_symbols in dwarf2read.c only descends into DIEs that have
a name; but anonymous namespaces can lead to DIEs without names which
have interesting children (where by "interesting" I mean that we want
to add partial symbols corresponding to them).  This could result in
us missing some partial symbols.

I've enclosed a patch below: for obvious reasons, it should go into
GDB 5.3 as well.  I've run GDB on a copy of GDB with this patch
applied and followed the control flow when GDB runs into various sorts
of DW_TAG_namespace entries, and the control flow seems to behave like
I expect it to.  (This is, admittedly, after only a limited amount of
testing.)  I'm in the middle of running the testsuite (I'll run it
twice, once on a compiler that doesn't generate DW_TAG_namespace
entries and once on a compiler that does); assuming that there are no
new regressions, is the patch okay?

David Carlton
carlton@math.stanford.edu

2002-11-21  David Carlton  <carlton@math.stanford.edu>

	* dwarf2read.c (scan_partial_symbols): Descend into namespace
	pdi's with no name.

Index: dwarf2read.c
===================================================================
RCS file: /cvs/src/src/gdb/dwarf2read.c,v
retrieving revision 1.75
diff -u -p -r1.75 dwarf2read.c
--- dwarf2read.c	11 Nov 2002 00:55:34 -0000	1.75
+++ dwarf2read.c	22 Nov 2002 00:11:49 -0000
@@ -1359,7 +1359,9 @@ scan_partial_symbols (char *info_ptr, st
     {
       info_ptr = read_partial_die (&pdi, abfd, info_ptr, cu_header);
 
-      if (pdi.name)
+      /* Anonymous namespaces have no name but are interesting.  */
+
+      if (pdi.name != NULL || pdi.tag == DW_TAG_namespace)
 	{
 	  switch (pdi.tag)
 	    {