diff options
author | Tom Tromey <tromey@redhat.com> | 2011-07-20 15:13:49 +0000 |
---|---|---|
committer | Tom Tromey <tromey@redhat.com> | 2011-07-20 15:13:49 +0000 |
commit | 6c83ed5236196bc958718f86d547551f25592659 (patch) | |
tree | bdb4cc06944139cb5b45d907e03d0d32555be813 /gdb | |
parent | 8b70b953ba15d6f6b1357d4e2186dca8a58bb4b2 (diff) | |
download | gdb-6c83ed5236196bc958718f86d547551f25592659.zip gdb-6c83ed5236196bc958718f86d547551f25592659.tar.gz gdb-6c83ed5236196bc958718f86d547551f25592659.tar.bz2 |
* dwarf2read.c (process_enumeration_scope): Do not call new_symbol
in some declaration-only cases.
Diffstat (limited to 'gdb')
-rw-r--r-- | gdb/ChangeLog | 5 | ||||
-rw-r--r-- | gdb/dwarf2read.c | 26 |
2 files changed, 31 insertions, 0 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index ca053d2..aa5bdf1 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,10 @@ 2011-07-20 Tom Tromey <tromey@redhat.com> + * dwarf2read.c (process_enumeration_scope): Do not call new_symbol + in some declaration-only cases. + +2011-07-18 Tom Tromey <tromey@redhat.com> + PR symtab/12984: * dwarf2read.c (dwarf2_section_info_def): New typedef. (struct dwarf2_per_objfile) <types>: Change to a VEC. diff --git a/gdb/dwarf2read.c b/gdb/dwarf2read.c index a373b09..64325cb 100644 --- a/gdb/dwarf2read.c +++ b/gdb/dwarf2read.c @@ -1207,6 +1207,11 @@ static struct die_info *follow_die_sig (struct die_info *, struct attribute *, struct dwarf2_cu **); +static struct signatured_type *lookup_signatured_type_at_offset + (struct objfile *objfile, + struct dwarf2_section_info *section, + unsigned int offset); + static void read_signatured_type_at_offset (struct objfile *objfile, struct dwarf2_section_info *sect, unsigned int offset); @@ -7730,6 +7735,27 @@ process_enumeration_scope (struct die_info *die, struct dwarf2_cu *cu) TYPE_UNSIGNED (this_type) = 1; } + /* If we are reading an enum from a .debug_types unit, and the enum + is a declaration, and the enum is not the signatured type in the + unit, then we do not want to add a symbol for it. Adding a + symbol would in some cases obscure the true definition of the + enum, giving users an incomplete type when the definition is + actually available. Note that we do not want to do this for all + enums which are just declarations, because C++0x allows forward + enum declarations. */ + if (cu->per_cu->debug_type_section + && die_is_declaration (die, cu)) + { + struct signatured_type *type_sig; + + type_sig + = lookup_signatured_type_at_offset (dwarf2_per_objfile->objfile, + cu->per_cu->debug_type_section, + cu->per_cu->offset); + if (type_sig->type_offset != die->offset) + return; + } + new_symbol (die, this_type, cu); } |