This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH v3 3/5] Code cleanup: dwarf2_initialize_objfile return value
- From: Pedro Alves <palves at redhat dot com>
- To: Yao Qi <qiyaoltc at gmail dot com>
- Cc: Jan Kratochvil <jan dot kratochvil at redhat dot com>, GDB Patches <gdb-patches at sourceware dot org>, Victor Leschuk <vleschuk at accesssoftek dot com>
- Date: Mon, 11 Dec 2017 13:59:56 +0000
- Subject: Re: [PATCH v3 3/5] Code cleanup: dwarf2_initialize_objfile return value
- Authentication-results: sourceware.org; auth=none
- References: <149790572259.20186.14601775821404892582.stgit@host1.jankratochvil.net> <149790575186.20186.4091613230201630619.stgit@host1.jankratochvil.net> <83d5d4a6-a137-b680-1af0-33b895bd817d@redhat.com> <CAH=s-PPpOHjwODDcj2_LLTFuFBCTYp5eh_vTCk0dySfjH+8kWg@mail.gmail.com> <653fdb35-4d09-283f-a2a2-cb6b0fb6d000@redhat.com>
On 12/11/2017 01:20 PM, Pedro Alves wrote:
> On 12/11/2017 01:03 PM, Yao Qi wrote:
>
>> This breaks the arm-wince-pe build,
>>
>> dwarf2read.o: In function `dwarf2_initialize_objfile(objfile*)':
>> /home/yao.qi/SourceCode/gnu/binutils-gdb/gdb/dwarf2read.c:6486:
>> undefined reference to `elf_sym_fns_gdb_index'
>> /home/yao.qi/SourceCode/gnu/binutils-gdb/gdb/dwarf2read.c:6490:
>> undefined reference to `elf_sym_fns_debug_names'
>> /home/yao.qi/SourceCode/gnu/binutils-gdb/gdb/dwarf2read.c:6495:
>> undefined reference to `elf_sym_fns_lazy_psyms'
>> collect2: error: ld returned 1 exit status
>> Makefile:1920: recipe for target 'gdb' failed
>>
>> https://ci.linaro.org/job/tcwg-binutils/4395/
>>
>
> Eh, looks like the bridge was closer than I realized... I forgot
> the non-elf ports don't include elfread.c in the build. So we
> really need to do something else here. Maybe an enum instead of
> the original boolean.
Something like this. I had already added a boolean for the index
variant to dwarf2read.c, so this reuses that.
I haven't tried to build for --target=arm-wince-pe yet, but this
probably works. I'll get back to this in a bit when I have a chance.
>From 223c6e730f2f85ca4732335964b616006974e078 Mon Sep 17 00:00:00 2001
From: Pedro Alves <palves@redhat.com>
Date: Mon, 11 Dec 2017 13:24:32 +0000
Subject: [PATCH] Unbreak non-ELF ports
---
gdb/dwarf2read.c | 32 ++++++++++++++------------------
gdb/elfread.c | 29 +++++++++++++++++++++++------
gdb/symfile.h | 17 ++++++++++++++++-
3 files changed, 53 insertions(+), 25 deletions(-)
diff --git a/gdb/dwarf2read.c b/gdb/dwarf2read.c
index 2aeb506..b4e60a4 100644
--- a/gdb/dwarf2read.c
+++ b/gdb/dwarf2read.c
@@ -6451,11 +6451,10 @@ const struct quick_symbol_functions dwarf2_debug_names_functions =
dw2_map_symbol_filenames
};
-/* Initialize for reading DWARF for this objfile. Return 0 if this
- file will use psymtabs, or 1 if using the GNU index. */
+/* See symfile.h. */
-const sym_fns &
-dwarf2_initialize_objfile (struct objfile *objfile)
+bool
+dwarf2_initialize_objfile (struct objfile *objfile, dw_index_kind *index_kind)
{
/* If we're about to read full symbols, don't bother with the
indices. In this case we also don't care if some other debug
@@ -6483,16 +6482,23 @@ 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 elf_sym_fns_gdb_index;
+ *index_kind = dw_index_kind::GDB_INDEX;
+ return true;
}
if (dwarf2_read_debug_names (objfile))
- return elf_sym_fns_debug_names;
+ {
+ *index_kind = dw_index_kind::DEBUG_NAMES;
+ return true;
+ }
if (dwarf2_read_index (objfile))
- return elf_sym_fns_gdb_index;
+ {
+ *index_kind = dw_index_kind::GDB_INDEX;
+ return true;
+ }
- return elf_sym_fns_lazy_psyms;
+ return false;
}
@@ -26799,16 +26805,6 @@ assert_file_size (FILE *file, const char *filename, size_t expected_size)
gdb_assert (file_size == expected_size);
}
-/* An index variant. */
-enum dw_index_kind
-{
- /* GDB's own .gdb_index format. */
- GDB_INDEX,
-
- /* DWARF5 .debug_names. */
- DEBUG_NAMES,
-};
-
/* Create an index file for OBJFILE in the directory DIR. */
static void
diff --git a/gdb/elfread.c b/gdb/elfread.c
index 31288a9..7a41c26 100644
--- a/gdb/elfread.c
+++ b/gdb/elfread.c
@@ -48,6 +48,11 @@
#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_debug_names;
+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. */
@@ -1211,11 +1216,25 @@ elf_symfile_read (struct objfile *objfile, symfile_add_flags symfile_flags)
if (dwarf2_has_info (objfile, NULL))
{
- /* elf_sym_fns_gdb_index cannot handle simultaneous non-DWARF debug
- information present in OBJFILE. If there is such debug info present
- never use .gdb_index. */
+ dw_index_kind index_kind;
- if (objfile_has_partial_symbols (objfile))
+ /* elf_sym_fns_gdb_index cannot handle simultaneous non-DWARF
+ debug information present in OBJFILE. If there is such debug
+ info present never use an index. */
+ if (!objfile_has_partial_symbols (objfile)
+ && dwarf2_initialize_objfile (objfile, &index_kind))
+ {
+ switch (index_kind)
+ {
+ case dw_index_kind::GDB_INDEX:
+ objfile_set_sym_fns (objfile, &elf_sym_fns_gdb_index);
+ break;
+ case dw_index_kind::DEBUG_NAMES:
+ objfile_set_sym_fns (objfile, &elf_sym_fns_debug_names);
+ break;
+ }
+ }
+ else
{
/* It is ok to do this even if the stabs reader made some
partial symbols, because OBJF_PSYMTABS_READ has not been
@@ -1223,8 +1242,6 @@ 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
diff --git a/gdb/symfile.h b/gdb/symfile.h
index e903c60..93fbe68 100644
--- a/gdb/symfile.h
+++ b/gdb/symfile.h
@@ -603,7 +603,22 @@ extern void dwarf2_get_section_info (struct objfile *,
asection **, const gdb_byte **,
bfd_size_type *);
-extern const sym_fns &dwarf2_initialize_objfile (struct objfile *);
+/* A DWARF names index variant. */
+enum class dw_index_kind
+{
+ /* GDB's own .gdb_index format. */
+ GDB_INDEX,
+
+ /* DWARF5 .debug_names. */
+ DEBUG_NAMES,
+};
+
+/* Initialize for reading DWARF for OBJFILE. Return false if this
+ file will use psymtabs, or true if using an index, in which case
+ *INDEX_KIND is set to the index variant in use. */
+extern bool dwarf2_initialize_objfile (struct objfile *objfile,
+ dw_index_kind *index_kind);
+
extern void dwarf2_build_psymtabs (struct objfile *);
extern void dwarf2_build_frame_info (struct objfile *);
--
2.5.5