[PATCH] GAS: Fix `abort' expansion in write.c

Alan Modra amodra@gmail.com
Wed Jun 29 00:02:00 GMT 2016


On Tue, Jun 28, 2016 at 05:33:38PM +0100, Maciej W. Rozycki wrote:
> With the removal of "libbfd.h" restore the latter message format.
> 
> 	gas/
> 	* write.c: Remove "libbfd.h" inclusion.

OK for master and binutils-2.26.

>  This has passed regression testing over my usual set of 162 targets so I 
> take it whatever the reason for the inclusion of "libbfd.h" was it is no 
> longer there and rather than thinking of a way to avoid the override made 
> by "libbfd.h" we can simply remove the offending inclusion.  No other 
> generic GAS source includes this header; target maintainers may want to 
> review their code though.

I was curious so checked out the 2007 source, and found that removing
libbfd.h gave:
/home/alan/src/binutils-gdb/gas/write.c: In function ‘set_symtab’:
/home/alan/src/binutils-gdb/gas/write.c:1326:16: warning: implicit declaration of function ‘bfd_alloc’ [-Wimplicit-function-declaration]
       asympp = bfd_alloc (stdoutput, amt);
                ^
bfd_alloc has since moved to bfd.h.

-- 
Alan Modra
Australia Development Lab, IBM



More information about the Binutils mailing list