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] GAS: Fix `abort' expansion in write.c


On Wed, 29 Jun 2016, Alan Modra 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.

 Applied and backported, thanks.

> >  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.

 Thanks for checking.  Perhaps backends can simply remove the inclusion 
then as well, with any interface definitions still required moved to 
"bfd.h"; it looks to me like "libbfd.h" is not really meant to be referred 
from outside BFD.

  Maciej


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