On Tue, Mar 24, 2020 at 3:38 AM Nick Clifton <nickc@redhat.com> wrote:
>
> Hi H.J.
>
> > bfd/
> >
> >       PR binutils/25708
> >       * elf-bfd.h (_bfd_elf_get_symbol_version_name): New.
> >       * elf.c (_bfd_elf_get_symbol_version_name): New function.  Based
> >       on the previous _bfd_elf_get_symbol_version_string.
> >       (_bfd_elf_get_symbol_version_string): Use it.
> >
> > binutils/
> >
> >       PR binutils/25708
> >       * nm.c (SYM_NAME): Removed.
> >       (print_symname): Add a pointer to struct extended_symbol_info
> >       argument.  Call _bfd_elf_get_symbol_version_name to get symbol
> >       version.
> >       (print_symdef_entry): Pass NULL to print_symname.
> >       (print_symbol_info_bsd): Update call to print_symname.
> >       (print_symbol_info_sysv): Likewise.
> >       (print_symbol_info_posix): Likewise.
> >
> > ld/
> >
> >       PR binutils/25708
> >       * testsuite/ld-elf/pr25707.d: New file.
>
> Approved - please apply.
>

We can't call _bfd_elf_get_symbol_version_name from nm.c since it isn't
available for all target configurations.  This patch add a bfd_boolean
argument to bfd_get_symbol_version_string instead.

OK for master?


-- 
H.J.