From ccefbec3745e5ec6eab196d8fd7a18e8fa740da9 Mon Sep 17 00:00:00 2001 From: Elena Zannoni Date: Thu, 19 Feb 2004 19:01:26 +0000 Subject: [PATCH] 2004-02-19 Joel Brobecker Committed by Elena Zannoni * symtab.c (find_pc_sect_psymtab): Return the psymtab that contains a symbol wich is the best, non-exact match for the given pc. Update comments. --- gdb/ChangeLog | 10 +++++++++- gdb/symtab.c | 41 ++++++++++++++++++++++++++++++++++++++--- 2 files changed, 47 insertions(+), 4 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 083fcaa..21d73d8 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,13 @@ -2004-02-19 Elena Zannoni +2004-02-19 Joel Brobecker + + Committed by Elena Zannoni + + * symtab.c (find_pc_sect_psymtab): Return the psymtab that + contains a symbol wich is the best, non-exact match for the given + pc. Update comments. +2004-02-19 Elena Zannoni + * event-top.c (async_request_quit): Remove uses of REQUEST_QUIT macro, which was part of the now removed Mach 3 port. * utils.c (request_quit): Ditto. diff --git a/gdb/symtab.c b/gdb/symtab.c index 99f138a..6995891b5 100644 --- a/gdb/symtab.c +++ b/gdb/symtab.c @@ -672,8 +672,10 @@ init_sal (struct symtab_and_line *sal) -/* Find which partial symtab on contains PC and SECTION. Return 0 if none. */ - +/* Find which partial symtab contains PC and SECTION. Return 0 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 (CORE_ADDR pc, asection *section) { @@ -698,6 +700,8 @@ find_pc_sect_psymtab (CORE_ADDR pc, asection *section) if (pc >= pst->textlow && pc < pst->texthigh) { struct partial_symtab *tpst; + struct partial_symtab *best_pst = pst; + struct partial_symbol *best_psym = NULL; /* An objfile that has its functions reordered might have many partial symbol tables containing the PC, but @@ -710,6 +714,13 @@ find_pc_sect_psymtab (CORE_ADDR pc, asection *section) if (msymbol == NULL) return (pst); + /* The code range of partial symtabs sometimes overlap, so, in + the loop below, we need to check all partial symtabs and + find the one that fits better for the given PC address. We + select the partial symtab that contains a symbol whose + address is closest to the PC address. By closest we mean + that find_pc_sect_symbol returns the symbol with address + that is closest and still less than the given PC. */ for (tpst = pst; tpst != NULL; tpst = tpst->next) { if (pc >= tpst->textlow && pc < tpst->texthigh) @@ -721,9 +732,33 @@ find_pc_sect_psymtab (CORE_ADDR pc, asection *section) && SYMBOL_VALUE_ADDRESS (p) == SYMBOL_VALUE_ADDRESS (msymbol)) return (tpst); + if (p != NULL) + { + /* We found a symbol in this partial symtab which + matches (or is closest to) PC, check whether it + is closer than our current BEST_PSYM. Since + this symbol address is necessarily lower or + equal to PC, the symbol closer to PC is the + symbol which address is the highest. */ + /* This way we return the psymtab which contains + such best match symbol. This can help in cases + where the symbol information/debuginfo is not + complete, like for instance on IRIX6 with gcc, + where no debug info is emitted for + statics. (See also the nodebug.exp + testcase.) */ + if (best_psym == NULL + || SYMBOL_VALUE_ADDRESS (p) + > SYMBOL_VALUE_ADDRESS (best_psym)) + { + best_psym = p; + best_pst = tpst; + } + } + } } - return (pst); + return (best_pst); } } return (NULL); -- 2.7.4