Change dwarf2_initialize_objfile's return value
authorJan Kratochvil <jan.kratochvil@redhat.com>
Fri, 8 Dec 2017 22:44:11 +0000 (22:44 +0000)
committerPedro Alves <palves@redhat.com>
Fri, 8 Dec 2017 23:37:30 +0000 (23:37 +0000)
dwarf2_initialize_objfile was returning boolean whether it is psymtabs
or .gdb_index while now it needs to return also whether it is
.debug_names.

gdb/ChangeLog
2017-12-08  Jan Kratochvil  <jan.kratochvil@redhat.com>

* defs.h (elf_sym_fns_lazy_psyms, elf_sym_fns_gdb_index): Move here
declarations from elfread.c.
(dwarf2_initialize_objfile): Change return value.
* elfread.c (elf_sym_fns_lazy_psyms, elf_sym_fns_gdb_index): Move these
declarations to defs.h.
(elf_symfile_read): Adjust dwarf2_initialize_objfile caller.
* symfile.h (dwarf2_initialize_objfile): Change return type.

gdb/ChangeLog
gdb/defs.h
gdb/dwarf2read.c
gdb/elfread.c
gdb/symfile.h

index 0f26785f16ca5b44d944eacafa3d6154f62b20c9..b6f37145abe400d11c5067ff0ab711d4f4834f77 100644 (file)
@@ -1,3 +1,13 @@
+2017-12-08  Jan Kratochvil  <jan.kratochvil@redhat.com>
+
+       * defs.h (elf_sym_fns_lazy_psyms, elf_sym_fns_gdb_index): Move here
+       declarations from elfread.c.
+       (dwarf2_initialize_objfile): Change return value.
+       * elfread.c (elf_sym_fns_lazy_psyms, elf_sym_fns_gdb_index): Move these
+       declarations to defs.h.
+       (elf_symfile_read): Adjust dwarf2_initialize_objfile caller.
+       * symfile.h (dwarf2_initialize_objfile): Change return type.
+
 2017-12-08  Jan Kratochvil  <jan.kratochvil@redhat.com>
            Pedro Alves  <palves@redhat.com>
 
index f76293fedf77d723f32b9706153a75c859e5d64a..a873a55e49bbb5cccee866643d8596350320d60a 100644 (file)
@@ -679,6 +679,11 @@ extern int (*deprecated_ui_load_progress_hook) (const char *section,
 extern void initialize_progspace (void);
 extern void initialize_inferiors (void);
 
+/* From elfread.c */
+
+extern const struct sym_fns elf_sym_fns_lazy_psyms;
+extern const struct sym_fns elf_sym_fns_gdb_index;
+
 /* * Special block numbers */
 
 enum block_enum
index a9daf1f3bc70bad589786961a76a292c32b4a858..798af72830575009e3ce895775691b68fd44c80f 100644 (file)
@@ -5291,7 +5291,7 @@ const struct quick_symbol_functions dwarf2_gdb_index_functions =
 /* Initialize for reading DWARF for this objfile.  Return 0 if this
    file will use psymtabs, or 1 if using the GNU index.  */
 
-int
+const sym_fns &
 dwarf2_initialize_objfile (struct objfile *objfile)
 {
   /* If we're about to read full symbols, don't bother with the
@@ -5320,13 +5320,13 @@ dwarf2_initialize_objfile (struct objfile *objfile)
       /* Return 1 so that gdb sees the "quick" functions.  However,
         these functions will be no-ops because we will have expanded
         all symtabs.  */
-      return 1;
+      return elf_sym_fns_gdb_index;
     }
 
   if (dwarf2_read_index (objfile))
-    return 1;
+    return elf_sym_fns_gdb_index;
 
-  return 0;
+  return elf_sym_fns_lazy_psyms;
 }
 
 \f
index b8881d92a7afc382a336158a78efb29c92b74b48..904696f9e97c722b10d5a61da84bf2ebdbe54f80 100644 (file)
 #include "location.h"
 #include "auxv.h"
 
-/* Forward declarations.  */
-extern const struct sym_fns elf_sym_fns_gdb_index;
-extern const struct sym_fns elf_sym_fns_lazy_psyms;
-
 /* The struct elfinfo is available only during ELF symbol table and
    psymtab reading.  It is destroyed at the completion of psymtab-reading.
    It's local to elf_symfile_read.  */
@@ -1219,10 +1215,7 @@ elf_symfile_read (struct objfile *objfile, symfile_add_flags symfile_flags)
         information present in OBJFILE.  If there is such debug info present
         never use .gdb_index.  */
 
-      if (!objfile_has_partial_symbols (objfile)
-         && dwarf2_initialize_objfile (objfile))
-       objfile_set_sym_fns (objfile, &elf_sym_fns_gdb_index);
-      else
+      if (objfile_has_partial_symbols (objfile))
        {
          /* It is ok to do this even if the stabs reader made some
             partial symbols, because OBJF_PSYMTABS_READ has not been
@@ -1230,6 +1223,8 @@ elf_symfile_read (struct objfile *objfile, symfile_add_flags symfile_flags)
             when needed.  */
          objfile_set_sym_fns (objfile, &elf_sym_fns_lazy_psyms);
        }
+      else
+       objfile_set_sym_fns (objfile, &dwarf2_initialize_objfile (objfile));
     }
   /* If the file has its own symbol tables it has no separate debug
      info.  `.dynsym'/`.symtab' go to MSYMBOLS, `.debug_info' goes to
index fa38abf49627e916d29ba780f22c544e338530a8..194dccf82239d49f22d35a56f6303b1c41dd7de9 100644 (file)
@@ -601,7 +601,7 @@ extern void dwarf2_get_section_info (struct objfile *,
                                     asection **, const gdb_byte **,
                                     bfd_size_type *);
 
-extern int dwarf2_initialize_objfile (struct objfile *);
+extern const sym_fns &dwarf2_initialize_objfile (struct objfile *);
 extern void dwarf2_build_psymtabs (struct objfile *);
 extern void dwarf2_build_frame_info (struct objfile *);