gdb, ada: update ada_lookup_simple_minsym
authorMarkus Metzger <markus.t.metzger@intel.com>
Tue, 12 Apr 2022 14:05:26 +0000 (16:05 +0200)
committerMarkus Metzger <markus.t.metzger@intel.com>
Tue, 18 Oct 2022 12:16:09 +0000 (14:16 +0200)
Iterate over objfile in search order using the objfile of the context
block as current_objfile so the iteration can stay inside the block's
linker namespace.

gdb/ada-exp.y
gdb/ada-lang.c
gdb/ada-lang.h

index f67c221d8733fd523294038e81b1c5c3b95adcfb..3ec9e4d610c20a5ef791008bc3bf6606271e22b4 100644 (file)
@@ -1698,8 +1698,15 @@ write_var_or_type (struct parser_state *par_state,
            }
          else if (syms.empty ())
            {
+             if (block == nullptr)
+               block = get_selected_block (nullptr);
+
+             struct objfile *objfile = nullptr;
+             if (block != nullptr)
+               objfile = block_objfile (block);
+
              struct bound_minimal_symbol msym
-               = ada_lookup_simple_minsym (decoded_name.c_str ());
+               = ada_lookup_simple_minsym (decoded_name.c_str (), objfile);
              if (msym.minsym != NULL)
                {
                  par_state->push_new<ada_var_msym_value_operation> (msym);
index f4d3f32d3397c124e6532534ce62e39c80a1c730..a24b1b5b8ef87c1b4db627239537a0594862cddf 100644 (file)
@@ -4901,7 +4901,7 @@ add_defn_to_vec (std::vector<struct block_symbol> &result,
    global symbols are searched.  */
 
 struct bound_minimal_symbol
-ada_lookup_simple_minsym (const char *name)
+ada_lookup_simple_minsym (const char *name, struct objfile *objfile)
 {
   struct bound_minimal_symbol result;
 
@@ -4911,19 +4911,23 @@ ada_lookup_simple_minsym (const char *name)
   symbol_name_matcher_ftype *match_name
     = ada_get_symbol_name_matcher (lookup_name);
 
-  for (objfile *objfile : current_program_space->objfiles ())
-    {
-      for (minimal_symbol *msymbol : objfile->msymbols ())
-       {
-         if (match_name (msymbol->linkage_name (), lookup_name, NULL)
-             && msymbol->type () != mst_solib_trampoline)
-           {
-             result.minsym = msymbol;
-             result.objfile = objfile;
-             break;
-           }
-       }
-    }
+  gdbarch_iterate_over_objfiles_in_search_order
+    (objfile != NULL ? objfile->arch () : target_gdbarch (),
+     [&result, lookup_name, match_name] (struct objfile *obj)
+       {
+        for (minimal_symbol *msymbol : obj->msymbols ())
+          {
+            if (match_name (msymbol->linkage_name (), lookup_name, nullptr)
+                && msymbol->type () != mst_solib_trampoline)
+              {
+                result.minsym = msymbol;
+                result.objfile = obj;
+                return 1;
+              }
+          }
+
+        return 0;
+       }, objfile);
 
   return result;
 }
index 84c691b94ee662ccdae6554c047adc1555e97bc3..d9c74f0a552d1b6f3315642ead757329fa3900da 100644 (file)
@@ -234,7 +234,8 @@ extern void ada_lookup_encoded_symbol
   (const char *name, const struct block *block, domain_enum domain,
    struct block_symbol *symbol_info);
 
-extern struct bound_minimal_symbol ada_lookup_simple_minsym (const char *);
+extern struct bound_minimal_symbol ada_lookup_simple_minsym (const char *,
+                                                            objfile *);
 
 extern int ada_scan_number (const char *, int, LONGEST *, int *);