From: Maciej W. Rozycki Date: Thu, 29 Mar 2018 13:09:48 +0000 (+0100) Subject: PR binutils/22875: MIPS: Remove duplicate unsupported relocation processing X-Git-Url: http://git.efficios.com/?a=commitdiff_plain;h=75def2abc3dafb52418405905cd49e9c107c2640;p=deliverable%2Fbinutils-gdb.git PR binutils/22875: MIPS: Remove duplicate unsupported relocation processing Remove a duplicate `unsupported relocation type' message and the setting of the `bfd_error_bad_value' error from `mips_elf32_rtype_to_howto', added with commit f3185997ac09 ("PR 22875: Stop strip corrupting unknown relocs"), . This message is already produced and the `bfd_error_bad_value' error set by `mips_elf32_rtype_to_howto' before a NULL howto is returned, so there is no need to repeat these actions here. bfd/ * elf32-mips.c (mips_info_to_howto_rel): Remove the calls to `_bfd_error_handler' and to set the `bfd_error_bad_value' error. --- diff --git a/bfd/ChangeLog b/bfd/ChangeLog index 21545593c7..925310e792 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,8 @@ +2018-03-29 Maciej W. Rozycki + + * elf32-mips.c (mips_info_to_howto_rel): Remove the calls to + `_bfd_error_handler' and to set the `bfd_error_bad_value' error. + 2018-03-28 Maciej W. Rozycki * elf-hppa.h (elf_hppa_info_to_howto_rel): Remove diff --git a/bfd/elf32-mips.c b/bfd/elf32-mips.c index 37676778e1..c3c96a4d6a 100644 --- a/bfd/elf32-mips.c +++ b/bfd/elf32-mips.c @@ -2239,13 +2239,9 @@ mips_info_to_howto_rel (bfd *abfd, arelent *cache_ptr, Elf_Internal_Rela *dst) r_type = ELF32_R_TYPE (dst->r_info); cache_ptr->howto = mips_elf32_rtype_to_howto (abfd, r_type, FALSE); + if (cache_ptr->howto == NULL) - { - /* xgettext:c-format */ - _bfd_error_handler (_("%pB: unsupported relocation type %#x"), abfd, r_type); - bfd_set_error (bfd_error_bad_value); - return FALSE; - } + return FALSE; /* The addend for a GPREL16 or LITERAL relocation comes from the GP value for the object file. We get the addend now, rather than