From: Tom Tromey Date: Sat, 20 Mar 2021 23:23:40 +0000 (-0600) Subject: Remove last objfile partial_symtab references from psymtab.c X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=3aa31ce7884f979c68d440a4352c8d69a3283f6c;p=binutils-gdb.git Remove last objfile partial_symtab references from psymtab.c This removes the last references to the partial_symtab via the objfile from psymtab.c. require_partial_symbols is now a method on psymbol_functions, and maintenance_print_psymbols is changed to use dynamic_cast to verify that it is examining partial symbols. gdb/ChangeLog 2021-03-20 Tom Tromey * psymtab.c (psymbol_functions::require_partial_symbols): Rename. (psymbol_functions::find_pc_sect_psymtab): Rename. (psymbol_functions::find_pc_sect_compunit_symtab) (maintenance_print_psymbols, maintenance_check_psymtabs): Update. * psympriv.h (struct psymbol_functions) : Declare new method. : Likewise. --- diff --git a/gdb/ChangeLog b/gdb/ChangeLog index bc746a1616a..3abf801bea0 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,13 @@ +2021-03-20 Tom Tromey + + * psymtab.c (psymbol_functions::require_partial_symbols): Rename. + (psymbol_functions::find_pc_sect_psymtab): Rename. + (psymbol_functions::find_pc_sect_compunit_symtab) + (maintenance_print_psymbols, maintenance_check_psymtabs): Update. + * psympriv.h (struct psymbol_functions) : + Declare new method. + : Likewise. + 2021-03-20 Tom Tromey * xcoffread.c (xcoff_start_psymtab): Add partial_symtabs parameter. diff --git a/gdb/psympriv.h b/gdb/psympriv.h index 8717bd259e1..85651f522d2 100644 --- a/gdb/psympriv.h +++ b/gdb/psympriv.h @@ -559,6 +559,18 @@ struct psymbol_functions : public quick_symbol_functions m_psymbol_map.clear (); } + /* Ensure the partial symbols for OBJFILE have been loaded. Return + a range adapter for the psymtabs. */ + psymtab_storage::partial_symtab_range require_partial_symbols + (struct objfile *objfile); + + /* Return the partial symbol storage associated with this + object. */ + const std::shared_ptr &get_partial_symtabs () const + { + return m_partial_symtabs; + } + /* Replace the partial symbol table storage in this object with SYMS. */ void set_partial_symtabs (const std::shared_ptr &syms) @@ -566,6 +578,17 @@ struct psymbol_functions : public quick_symbol_functions m_partial_symtabs = syms; } + /* Find which partial symtab contains PC and SECTION. Return NULL if + none. We return the psymtab that contains a symbol whose address + exactly matches PC, or, if we cannot find an exact match, the + psymtab that contains a symbol whose address is closest to PC. */ + + struct partial_symtab *find_pc_sect_psymtab + (struct objfile *objfile, + CORE_ADDR pc, + struct obj_section *section, + struct bound_minimal_symbol msymbol); + private: void fill_psymbol_map (struct objfile *objfile, diff --git a/gdb/psymtab.c b/gdb/psymtab.c index e662ba16cd6..b2f929912ea 100644 --- a/gdb/psymtab.c +++ b/gdb/psymtab.c @@ -80,11 +80,11 @@ psymtab_storage::install_psymtab (partial_symtab *pst) returns a range adapter suitable for iterating over the psymtabs of OBJFILE. */ -static psymtab_storage::partial_symtab_range -require_partial_symbols (struct objfile *objfile) +psymtab_storage::partial_symtab_range +psymbol_functions::require_partial_symbols (struct objfile *objfile) { objfile->require_partial_symbols (true); - return objfile->psymtabs (); + return m_partial_symtabs->range (); } /* Helper function for psym_map_symtabs_matching_filename that @@ -255,17 +255,13 @@ find_pc_sect_psymtab_closer (struct objfile *objfile, return best_pst; } -/* Find which partial symtab contains PC and SECTION. Return NULL if - none. We return the psymtab that contains a symbol whose address - exactly matches PC, or, if we cannot find an exact match, the - psymtab that contains a symbol whose address is closest to PC. */ +/* See psympriv.h. */ -static struct partial_symtab * -find_pc_sect_psymtab (struct objfile *objfile, - psymtab_storage *partial_symtabs, - CORE_ADDR pc, - struct obj_section *section, - struct bound_minimal_symbol msymbol) +struct partial_symtab * +psymbol_functions::find_pc_sect_psymtab (struct objfile *objfile, + CORE_ADDR pc, + struct obj_section *section, + struct bound_minimal_symbol msymbol) { /* Try just the PSYMTABS_ADDRMAP mapping first as it has better granularity than the later used TEXTLOW/TEXTHIGH one. However, we need @@ -279,14 +275,14 @@ find_pc_sect_psymtab (struct objfile *objfile, partial symtabs then we will end up returning a pointer to an object that is not a partial_symtab, which doesn't end well. */ - if (partial_symtabs->psymtabs != NULL - && partial_symtabs->psymtabs_addrmap != NULL) + if (m_partial_symtabs->psymtabs != NULL + && m_partial_symtabs->psymtabs_addrmap != NULL) { CORE_ADDR baseaddr = objfile->text_section_offset (); struct partial_symtab *pst = ((struct partial_symtab *) - addrmap_find (partial_symtabs->psymtabs_addrmap, + addrmap_find (m_partial_symtabs->psymtabs_addrmap, pc - baseaddr)); if (pst != NULL) { @@ -356,7 +352,6 @@ psymbol_functions::find_pc_sect_compunit_symtab int warn_if_readin) { struct partial_symtab *ps = find_pc_sect_psymtab (objfile, - m_partial_symtabs.get (), pc, section, msymbol); if (ps != NULL) @@ -1815,7 +1810,12 @@ maintenance_print_psymbols (const char *args, int from_tty) if (!print_for_objfile) continue; - psymtab_storage *partial_symtabs = objfile->partial_symtabs.get (); + psymbol_functions *psf + = dynamic_cast (objfile->qf.get ()); + if (psf == nullptr) + continue; + + psymtab_storage *partial_symtabs = psf->get_partial_symtabs ().get (); if (address_arg != NULL) { @@ -1824,8 +1824,7 @@ maintenance_print_psymbols (const char *args, int from_tty) /* We don't assume each pc has a unique objfile (this is for debugging). */ struct partial_symtab *ps - = find_pc_sect_psymtab (objfile, partial_symtabs, pc, - section, msymbol); + = psf->find_pc_sect_psymtab (objfile, pc, section, msymbol); if (ps != NULL) { if (!printed_objfile_header) @@ -1841,7 +1840,7 @@ maintenance_print_psymbols (const char *args, int from_tty) } else { - for (partial_symtab *ps : require_partial_symbols (objfile)) + for (partial_symtab *ps : psf->require_partial_symbols (objfile)) { int print_for_source = 0; @@ -1905,7 +1904,11 @@ maintenance_info_psymtabs (const char *regexp, int from_tty) actually find a symtab whose name matches. */ int printed_objfile_start = 0; - for (partial_symtab *psymtab : require_partial_symbols (objfile)) + psymbol_functions *psf + = dynamic_cast (objfile->qf.get ()); + if (psf == nullptr) + continue; + for (partial_symtab *psymtab : psf->require_partial_symbols (objfile)) { QUIT; @@ -2005,89 +2008,96 @@ maintenance_check_psymtabs (const char *ignore, int from_tty) const struct block *b; for (objfile *objfile : current_program_space->objfiles ()) - for (partial_symtab *ps : require_partial_symbols (objfile)) - { - struct gdbarch *gdbarch = objfile->arch (); + { + psymbol_functions *psf + = dynamic_cast (objfile->qf.get ()); + if (psf == nullptr) + continue; - /* We don't call psymtab_to_symtab here because that may cause symtab - expansion. When debugging a problem it helps if checkers leave - things unchanged. */ - cust = ps->get_compunit_symtab (objfile); + for (partial_symtab *ps : psf->require_partial_symbols (objfile)) + { + struct gdbarch *gdbarch = objfile->arch (); - /* First do some checks that don't require the associated symtab. */ - if (ps->text_high (objfile) < ps->text_low (objfile)) - { - printf_filtered ("Psymtab "); - puts_filtered (ps->filename); - printf_filtered (" covers bad range "); - fputs_filtered (paddress (gdbarch, ps->text_low (objfile)), - gdb_stdout); - printf_filtered (" - "); - fputs_filtered (paddress (gdbarch, ps->text_high (objfile)), - gdb_stdout); - printf_filtered ("\n"); - continue; - } + /* We don't call psymtab_to_symtab here because that may cause symtab + expansion. When debugging a problem it helps if checkers leave + things unchanged. */ + cust = ps->get_compunit_symtab (objfile); - /* Now do checks requiring the associated symtab. */ - if (cust == NULL) - continue; - bv = COMPUNIT_BLOCKVECTOR (cust); - b = BLOCKVECTOR_BLOCK (bv, STATIC_BLOCK); - for (partial_symbol *psym : ps->static_psymbols) - { - /* Skip symbols for inlined functions without address. These may - or may not have a match in the full symtab. */ - if (psym->aclass == LOC_BLOCK - && psym->ginfo.value.address == 0) + /* First do some checks that don't require the associated symtab. */ + if (ps->text_high (objfile) < ps->text_low (objfile)) + { + printf_filtered ("Psymtab "); + puts_filtered (ps->filename); + printf_filtered (" covers bad range "); + fputs_filtered (paddress (gdbarch, ps->text_low (objfile)), + gdb_stdout); + printf_filtered (" - "); + fputs_filtered (paddress (gdbarch, ps->text_high (objfile)), + gdb_stdout); + printf_filtered ("\n"); continue; + } - sym = block_lookup_symbol (b, psym->ginfo.search_name (), - symbol_name_match_type::SEARCH_NAME, - psym->domain); - if (!sym) - { - printf_filtered ("Static symbol `"); - puts_filtered (psym->ginfo.linkage_name ()); - printf_filtered ("' only found in "); - puts_filtered (ps->filename); - printf_filtered (" psymtab\n"); - } - } - b = BLOCKVECTOR_BLOCK (bv, GLOBAL_BLOCK); - for (partial_symbol *psym : ps->global_psymbols) - { - sym = block_lookup_symbol (b, psym->ginfo.search_name (), - symbol_name_match_type::SEARCH_NAME, - psym->domain); - if (!sym) - { - printf_filtered ("Global symbol `"); - puts_filtered (psym->ginfo.linkage_name ()); - printf_filtered ("' only found in "); - puts_filtered (ps->filename); - printf_filtered (" psymtab\n"); - } - } - if (ps->raw_text_high () != 0 - && (ps->text_low (objfile) < BLOCK_START (b) - || ps->text_high (objfile) > BLOCK_END (b))) - { - printf_filtered ("Psymtab "); - puts_filtered (ps->filename); - printf_filtered (" covers "); - fputs_filtered (paddress (gdbarch, ps->text_low (objfile)), - gdb_stdout); - printf_filtered (" - "); - fputs_filtered (paddress (gdbarch, ps->text_high (objfile)), - gdb_stdout); - printf_filtered (" but symtab covers only "); - fputs_filtered (paddress (gdbarch, BLOCK_START (b)), gdb_stdout); - printf_filtered (" - "); - fputs_filtered (paddress (gdbarch, BLOCK_END (b)), gdb_stdout); - printf_filtered ("\n"); - } - } + /* Now do checks requiring the associated symtab. */ + if (cust == NULL) + continue; + bv = COMPUNIT_BLOCKVECTOR (cust); + b = BLOCKVECTOR_BLOCK (bv, STATIC_BLOCK); + for (partial_symbol *psym : ps->static_psymbols) + { + /* Skip symbols for inlined functions without address. These may + or may not have a match in the full symtab. */ + if (psym->aclass == LOC_BLOCK + && psym->ginfo.value.address == 0) + continue; + + sym = block_lookup_symbol (b, psym->ginfo.search_name (), + symbol_name_match_type::SEARCH_NAME, + psym->domain); + if (!sym) + { + printf_filtered ("Static symbol `"); + puts_filtered (psym->ginfo.linkage_name ()); + printf_filtered ("' only found in "); + puts_filtered (ps->filename); + printf_filtered (" psymtab\n"); + } + } + b = BLOCKVECTOR_BLOCK (bv, GLOBAL_BLOCK); + for (partial_symbol *psym : ps->global_psymbols) + { + sym = block_lookup_symbol (b, psym->ginfo.search_name (), + symbol_name_match_type::SEARCH_NAME, + psym->domain); + if (!sym) + { + printf_filtered ("Global symbol `"); + puts_filtered (psym->ginfo.linkage_name ()); + printf_filtered ("' only found in "); + puts_filtered (ps->filename); + printf_filtered (" psymtab\n"); + } + } + if (ps->raw_text_high () != 0 + && (ps->text_low (objfile) < BLOCK_START (b) + || ps->text_high (objfile) > BLOCK_END (b))) + { + printf_filtered ("Psymtab "); + puts_filtered (ps->filename); + printf_filtered (" covers "); + fputs_filtered (paddress (gdbarch, ps->text_low (objfile)), + gdb_stdout); + printf_filtered (" - "); + fputs_filtered (paddress (gdbarch, ps->text_high (objfile)), + gdb_stdout); + printf_filtered (" but symtab covers only "); + fputs_filtered (paddress (gdbarch, BLOCK_START (b)), gdb_stdout); + printf_filtered (" - "); + fputs_filtered (paddress (gdbarch, BLOCK_END (b)), gdb_stdout); + printf_filtered ("\n"); + } + } + } } void _initialize_psymtab ();