This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH] GAS: Fix `abort' expansion in write.c
- From: "Maciej W. Rozycki" <macro at imgtec dot com>
- To: Alan Modra <amodra at gmail dot com>
- Cc: <binutils at sourceware dot org>
- Date: Wed, 29 Jun 2016 03:37:23 +0100
- Subject: Re: [PATCH] GAS: Fix `abort' expansion in write.c
- Authentication-results: sourceware.org; auth=none
- References: <alpine dot DEB dot 2 dot 00 dot 1606281625390 dot 4103 at tp dot orcam dot me dot uk> <20160629000207 dot GM3665 at bubble dot grove dot modra dot org>
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