From bfb05775e68c415e976761e14608485facfaefcf Mon Sep 17 00:00:00 2001 From: Jan Kratochvil Date: Thu, 24 May 2012 22:14:36 +0000 Subject: [PATCH] gdb/ * psymtab.c (lookup_symbol_aux_psymtabs): New variable stab_best. Use it as a fallback for TYPE_IS_OPAQUE. * symfile.h (struct quick_symbol_functions): Mention TYPE_OPAQUE symbols for lookup_symbol. gdb/testsuite/ * gdb.dwarf2/dw2-icc-opaque.S: Add debug_info_seg3 and .debug_abbrev_seg3. --- gdb/ChangeLog | 7 ++++ gdb/psymtab.c | 10 ++++- gdb/symfile.h | 4 +- gdb/testsuite/ChangeLog | 5 +++ gdb/testsuite/gdb.dwarf2/dw2-icc-opaque.S | 51 +++++++++++++++++++++++ 5 files changed, 74 insertions(+), 3 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 1e4e1ac107d..10f1ce8dc57 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,10 @@ +2012-05-24 Jan Kratochvil + + * psymtab.c (lookup_symbol_aux_psymtabs): New variable stab_best. Use + it as a fallback for TYPE_IS_OPAQUE. + * symfile.h (struct quick_symbol_functions): Mention TYPE_OPAQUE + symbols for lookup_symbol. + 2012-05-24 John Steele Scott PR symtab/13277: Resolving opaque structures in ICC generated binaries. diff --git a/gdb/psymtab.c b/gdb/psymtab.c index e463fff0bc8..647368c800d 100644 --- a/gdb/psymtab.c +++ b/gdb/psymtab.c @@ -510,6 +510,7 @@ lookup_symbol_aux_psymtabs (struct objfile *objfile, { struct partial_symtab *ps; const int psymtab_index = (block_index == GLOBAL_BLOCK ? 1 : 0); + struct symtab *stab_best = NULL; ALL_OBJFILE_PSYMTABS_REQUIRED (objfile, ps) { @@ -530,13 +531,18 @@ lookup_symbol_aux_psymtabs (struct objfile *objfile, } if (sym && strcmp_iw (SYMBOL_SEARCH_NAME (sym), name) == 0) - return stab; + { + if (!TYPE_IS_OPAQUE (SYMBOL_TYPE (sym))) + return stab; + + stab_best = stab; + } /* Keep looking through other psymtabs. */ } } - return NULL; + return stab_best; } /* Look in PST for a symbol in DOMAIN whose name matches NAME. Search diff --git a/gdb/symfile.h b/gdb/symfile.h index 8e837415ee2..ba68d4a8057 100644 --- a/gdb/symfile.h +++ b/gdb/symfile.h @@ -182,7 +182,9 @@ struct quick_symbol_functions indicates what sort of symbol to search for. Returns the newly-expanded symbol table in which the symbol is - defined, or NULL if no such symbol table exists. */ + defined, or NULL if no such symbol table exists. If OBJFILE + contains !TYPE_OPAQUE symbol prefer its symtab. If it contains + only TYPE_OPAQUE symbol(s), return at least that symtab. */ struct symtab *(*lookup_symbol) (struct objfile *objfile, int kind, const char *name, domain_enum domain); diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index fd97ca976fd..9b8afca3436 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2012-05-24 Jan Kratochvil + + * gdb.dwarf2/dw2-icc-opaque.S: Add debug_info_seg3 and + .debug_abbrev_seg3. + 2012-05-24 John Steele Scott PR symtab/13277: Resolving opaque structures in ICC generated binaries. diff --git a/gdb/testsuite/gdb.dwarf2/dw2-icc-opaque.S b/gdb/testsuite/gdb.dwarf2/dw2-icc-opaque.S index 7564fbad219..117866d5cb8 100644 --- a/gdb/testsuite/gdb.dwarf2/dw2-icc-opaque.S +++ b/gdb/testsuite/gdb.dwarf2/dw2-icc-opaque.S @@ -205,3 +205,54 @@ debug_info_seg2_end: .byte 0x08 /* DW_FORM_string. */ .2byte 0x0000 /* End abbrev. */ .byte 0x00 /* End abbrev table. */ + + + .file "opaque-pointer2.c" + + .section .debug_info + .align 1 +debug_info_seg3: + .4byte debug_info_seg3_end - 1f /* Length of compilation unit info. */ +1: + .2byte 0x0002 /* DWARF version number. */ + .4byte .debug_abbrev_seg3 /* Points to abbrev section for this unit. */ + .byte 0x04 /* Target address size. */ + + .byte 0x01 /* DIE 1: DW_TAG_compile_unit. */ + .byte 0x01 /* DW_AT_language = DW_LANG_C89. */ + .ascii "Intel(R) C Intel(R) 64 Compiler XE " /* DW_AT_producer. */ + .ascii "for applications running on Intel(R) 64, " + .ascii "Version 12.0.4.191 Build 20110427\n " + .asciz "Fixes SameLinkageName MemberPointers" + + .byte 0x04 /* DIE 4: DW_TAG_structure_type. */ + .byte 0x01 /* DW_AT_accessibility. */ + .byte 0x00 /* DW_AT_byte_size. */ + .asciz "opaque_struct_t" /* DW_AT_name. */ + + .byte 0x00 /* End DIE 1. */ +debug_info_seg3_end: + + .section .debug_abbrev +.debug_abbrev_seg3: + .align 1 + .byte 0x01 /* Abbrev 1. */ + .byte 0x11 /* DW_TAG_compile_unit. */ + .byte 0x01 /* DW_CHILDREN_yes. */ + .byte 0x13 /* DW_AT_language. */ + .byte 0x0b /* DW_FORM_data1. */ + .byte 0x25 /* DW_AT_producer. */ + .byte 0x08 /* DW_AT_string. */ + .2byte 0x0000 /* End abbrev. */ + + .byte 0x04 /* Abbrev 4. */ + .byte 0x13 /* DW_TAG_structure_type. */ + .byte 0x00 /* DW_CHILDREN_no. */ + .byte 0x32 /* DW_AT_accessibility. */ + .byte 0x0b /* DW_FORM_data1. */ + .byte 0x0b /* DW_AT_byte_size. */ + .byte 0x0b /* DW_FORM_data1. */ + .byte 0x03 /* DW_AT_name. */ + .byte 0x08 /* DW_FORM_string. */ + .2byte 0x0000 /* End abbrev. */ + .byte 0x00 /* End abbrev table. */ -- 2.30.2