[binutils-gdb] Convert lookup_symbol_in_objfile

Tom Tromey tromey@sourceware.org
Wed Sep 10 22:19:37 GMT 2025


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=86ac8c546235a67d6a6bb29476a3a9ac8f7a620a

commit 86ac8c546235a67d6a6bb29476a3a9ac8f7a620a
Author: Tom Tromey <tom@tromey.com>
Date:   Thu Jan 2 15:28:18 2025 -0700

    Convert lookup_symbol_in_objfile
    
    This converts lookup_symbol_in_objfile to the callback approach by
    removing the call to lookup_symbol_in_objfile_symtabs.  (The latter is
    not removed as there are still other callers.)
    
    Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=16994
    Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=16998
    Acked-By: Simon Marchi <simon.marchi@efficios.com>

Diff:
---
 gdb/symtab.c | 16 ++--------------
 1 file changed, 2 insertions(+), 14 deletions(-)

diff --git a/gdb/symtab.c b/gdb/symtab.c
index e039e43acc7..25b6a84fa92 100644
--- a/gdb/symtab.c
+++ b/gdb/symtab.c
@@ -2586,24 +2586,12 @@ lookup_symbol_in_objfile (struct objfile *objfile, enum block_enum block_index,
 			      ? "GLOBAL_BLOCK" : "STATIC_BLOCK",
 			      name, domain_name (domain).c_str ());
 
-  result = lookup_symbol_in_objfile_symtabs (objfile, block_index,
-					     name, domain);
-  if (result.symbol != NULL)
-    {
-      symbol_lookup_debug_printf
-	("lookup_symbol_in_objfile (...) = %s (in symtabs)",
-	 host_address_to_string (result.symbol));
-      return result;
-    }
-
   result = lookup_symbol_via_quick_fns (objfile, block_index,
 					name, domain);
-  symbol_lookup_debug_printf ("lookup_symbol_in_objfile (...) = %s%s",
+  symbol_lookup_debug_printf ("lookup_symbol_in_objfile (...) = %s",
 			      result.symbol != NULL
 			      ? host_address_to_string (result.symbol)
-			      : "NULL",
-			      result.symbol != NULL ? " (via quick fns)"
-			      : "");
+			      : "NULL");
   return result;
 }


More information about the Gdb-cvs mailing list