This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH v3 0/4] Allow loading symbol files with an offset
- From: Simon Marchi <simon dot marchi at ericsson dot com>
- To: Petr Tesarik <ptesarik at suse dot cz>, <gdb-patches at sourceware dot org>
- Cc: Simon Marchi <simon dot marchi at polymtl dot ca>, Eli Zaretskii <eliz at gnu dot org>
- Date: Wed, 27 Jun 2018 08:56:46 -0400
- Subject: Re: [PATCH v3 0/4] Allow loading symbol files with an offset
- References: <20180611120835.27343-1-ptesarik@suse.cz> <20180627121429.16414-1-ptesarik@suse.cz>
On 2018-06-27 08:14 AM, Petr Tesarik wrote:
> Implement an "-o" option for symbol-file and add-symbol-file.
>
> Changes since v2:
> * Fix the error message if add-symbol-file is called without arguments
> * Adjust texinfo description of add-symbol-file
> * Clarify the use of other_sections::sectindex
> * Use '.empty()' instead of '.size() == 0'
> * Split off set_objfile_default_section_offset
>
> Changes since v1:
> * Adjust help texts
> * Update gdb/NEWS
>
> Petr Tesarik (4):
> Add an optional offset option to the "symbol-file" command
> Make add-symbol-file's address argument optional
> Make sure that sorting does not change section order
> Add an optional offset option to the "add-symbol-file" command
>
> gdb/ChangeLog | 24 ++++++
> gdb/NEWS | 10 +++
> gdb/doc/ChangeLog | 13 ++++
> gdb/doc/gdb.texinfo | 28 ++++---
> gdb/symfile.c | 141 ++++++++++++++++++++++++++++++------
> gdb/symfile.h | 4 +-
> gdb/testsuite/ChangeLog | 13 ++++
> gdb/testsuite/gdb.base/relocate.exp | 95 ++++++++++++++++++++++++
> 8 files changed, 294 insertions(+), 34 deletions(-)
>
Thanks Petr, this version LGTM.
Simon