This is the mail archive of the
mailing list for the GDB project.
RFA: make it simpler to try debug_pubnames
- From: Tom Tromey <tromey at redhat dot com>
- To: gdb-patches at sourceware dot org
- Date: Thu, 31 Jul 2008 13:22:29 -0600
- Subject: RFA: make it simpler to try debug_pubnames
- Reply-to: tromey at redhat dot com
I wanted to try the debug_pubnames support today, but when I enabled
this I found that it did not compile, and when I made it compile, it
caused an internal error.
This patch fixes the compilation error and the crash, and makes it
simple to try this code by introducing a new symbol, which is used
rather than "#if 0".
The reason I would like this in the repository is to make it simpler
for other folks to try this out.
Please check the comment before WANT_PUBNAMES. I am not sure this is
actually an accurate assessment; my searches didn't turn up anything
Built on x86 F8. I did not regression-test this, because it does not
actually change any behavior.
2008-07-31 Tom Tromey <email@example.com>
* dwarf2read.c (WANT_PUBNAMES): New define.
(dwarf2_build_psymtabs): Use WANT_PUBNAMES. Check
dwarf_aranges_section and dwarf_pubnames_section.
(dwarf2_build_psymtabs_easy): Use WANT_PUBNAMES. Initialize
diff --git a/gdb/dwarf2read.c b/gdb/dwarf2read.c
index 8f1062d..ac97cd7 100644
@@ -69,6 +69,13 @@
can be used for any other data associated to the objfile (symbol
names, type names, location expressions to name a few). */
+/* Define this to 1 if you want to try out the debug_pubnames
+ support. This is disabled for the time being because it leads to a
+ change of behavior -- 'static' file-scoped variables are not
+ mentioned in debug_pubnames, but historically these do appear in
+ gdb's psymtab. */
+#define WANT_PUBNAMES 0
/* .debug_info header for a compilation unit
Because of alignment constraints, this structure has padding and cannot
@@ -734,7 +741,7 @@ dwarf2_invalid_attrib_class_complaint (const char *arg1, const char *arg2)
static void dwarf2_locate_sections (bfd *, asection *, void *);
static void dwarf2_build_psymtabs_easy (struct objfile *, int);
@@ -1258,8 +1265,8 @@ dwarf2_build_psymtabs (struct objfile *objfile, int mainline)
init_psymbol_list (objfile, 1024);
- if (dwarf_aranges_offset && dwarf_pubnames_offset)
+ if (dwarf_aranges_section && dwarf_pubnames_section)
/* Things are significantly easier if we have .debug_aranges and
.debug_pubnames sections */
@@ -1275,7 +1282,7 @@ dwarf2_build_psymtabs (struct objfile *objfile, int mainline)
/* Build the partial symbol table from the information in the
.debug_pubnames and .debug_aranges sections. */
@@ -1295,6 +1302,7 @@ dwarf2_build_psymtabs_easy (struct objfile *objfile, int mainline)
struct comp_unit_head cu_header;
unsigned int bytes_read;
+ cu_header.initial_length_size = 0;
entry_length = read_initial_length (abfd, pubnames_ptr, &cu_header,
pubnames_ptr += bytes_read;