This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH 5/6] PR binutils/22875: i860/ELF: Report unsupported relocation types


On Thu, Mar 29, 2018 at 06:34:31PM +0100, Maciej W. Rozycki wrote:
> Complement commit f3185997ac09 ("PR 22875: Stop strip corrupting unknown 
> relocs"), <https://sourceware.org/ml/binutils/2018-02/msg00445.html>, 
> and also set the `bfd_error_bad_value' error and report an unsupported 
> relocation type if a howto lookup fails with the i860 backend, fixing a 
> confusing `no error' error message and removing a binutils test failure:
> 
> failed with: <.../binutils/strip-new: tmpdir/bintest.o: no error>, expected: <.* bad value>
> .../binutils/strip-new: tmpdir/bintest.o: no error
> FAIL: binutils-all/strip-13
> 
> with the `i860-stardent-elf' target.
> 
> 	bfd/
> 	* elf32-i860.c (lookup_howto): Add `abfd' parameter.  Set the 
> 	`bfd_error_bad_value' error and call `_bfd_error_handler' on a 
> 	howto lookup failure.
> 	(elf32_i860_reloc_type_lookup): Adjust `lookup_howto' call 
> 	accordingly.
> 	(elf32_i860_info_to_howto_rela): Likewise.
> 	(elf32_i860_relocate_splitn): Likewise.
> 	(elf32_i860_relocate_pc16): Likewise.
> 	(elf32_i860_relocate_pc26): Likewise.
> 	(elf32_i860_relocate_section): Likewise.

OK, and thanks for writing this patch series.

Huh, I see i860-* has been obsolete since 2016-09-27.  So we have been
a little slow to carry out the "Support will be REMOVED in the next
major release of BINUTILS" threat.

-- 
Alan Modra
Australia Development Lab, IBM


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]