From: Doug Evans <xdje42@gmail.com>
Date: Sat, 15 Nov 2014 16:05:22 +0000 (-0800)
Subject: expand_symtab_containing_pc: Renamed from find_pc_sect_symtab_via_partial.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=2097ae2584c1018a7a58d93d9edd7f6d1d9654c8;p=binutils-gdb.git

expand_symtab_containing_pc: Renamed from find_pc_sect_symtab_via_partial.

This patch just renames one function.
Its only caller is in stack.c where we're printing a backtrace
with non-zero info_verbose and we want to make sure all the needed
symtabs are expanded before printing the backtrace
so that debug symbol reading messages don't pollute the backtrace.

I think the new name of the function makes clearer to the reader
what is going on.

gdb/ChangeLog:

	* symtab.c (expand_symtab_containing_pc): Renamed from
	find_pc_sect_symtab_via_partial.  All callers updated.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 3b9ba33032d..6198d5433c4 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2014-11-15  Doug Evans  <xdje42@gmail.com>
+
+	* symtab.c (expand_symtab_containing_pc): Renamed from
+	find_pc_sect_symtab_via_partial.  All callers updated.
+
 2014-11-15  Yao Qi  <yao@codesourcery.com>
 
 	* go32-nat.c (go32_create_inferior): Add missing parenthesis.
diff --git a/gdb/stack.c b/gdb/stack.c
index 4b34e0b8c59..35f491dfb34 100644
--- a/gdb/stack.c
+++ b/gdb/stack.c
@@ -1776,7 +1776,7 @@ backtrace_command_1 (char *count_exp, int show_locals, int no_filters,
 
 	  QUIT;
 	  pc = get_frame_address_in_block (fi);
-	  find_pc_sect_symtab_via_partial (pc, find_pc_mapped_section (pc));
+	  expand_symtab_containing_pc (pc, find_pc_mapped_section (pc));
 	}
     }
 
diff --git a/gdb/symtab.c b/gdb/symtab.c
index 4f4ecf81e87..4672ac31eab 100644
--- a/gdb/symtab.c
+++ b/gdb/symtab.c
@@ -1045,8 +1045,10 @@ matching_obj_sections (struct obj_section *obj_first,
   return 0;
 }
 
-struct symtab *
-find_pc_sect_symtab_via_partial (CORE_ADDR pc, struct obj_section *section)
+/* See symtab.h.  */
+
+void
+expand_symtab_containing_pc (CORE_ADDR pc, struct obj_section *section)
 {
   struct objfile *objfile;
   struct bound_minimal_symbol msymbol;
@@ -1061,20 +1063,18 @@ find_pc_sect_symtab_via_partial (CORE_ADDR pc, struct obj_section *section)
 	  || MSYMBOL_TYPE (msymbol.minsym) == mst_abs
 	  || MSYMBOL_TYPE (msymbol.minsym) == mst_file_data
 	  || MSYMBOL_TYPE (msymbol.minsym) == mst_file_bss))
-    return NULL;
+    return;
 
   ALL_OBJFILES (objfile)
   {
-    struct symtab *result = NULL;
+    struct symtab *s = NULL;
 
     if (objfile->sf)
-      result = objfile->sf->qf->find_pc_sect_symtab (objfile, msymbol,
-						     pc, section, 0);
-    if (result)
-      return result;
+      s = objfile->sf->qf->find_pc_sect_symtab (objfile, msymbol,
+						pc, section, 0);
+    if (s != NULL)
+      return;
   }
-
-  return NULL;
 }
 
 /* Debug symbols usually don't have section information.  We need to dig that
diff --git a/gdb/symtab.h b/gdb/symtab.h
index d78b832cf9d..d69fbcfc320 100644
--- a/gdb/symtab.h
+++ b/gdb/symtab.h
@@ -1163,10 +1163,9 @@ extern int find_pc_partial_function (CORE_ADDR, const char **, CORE_ADDR *,
 
 extern void clear_pc_function_cache (void);
 
-/* lookup partial symbol table by address and section.  */
+/* Expand symtab containing PC, SECTION if not already expanded.  */
 
-extern struct symtab *find_pc_sect_symtab_via_partial (CORE_ADDR,
-						       struct obj_section *);
+extern void expand_symtab_containing_pc (CORE_ADDR, struct obj_section *);
 
 /* lookup full symbol table by address.  */