Re: Pack Elf_External_Note

[Daniel Jacobowitz <drow at mvista dot com>]

On Thu, Jan 29, 2004 at 02:02:08PM -0500, Ian Lance Taylor wrote:
> Daniel Jacobowitz <drow@mvista.com> writes:
> 
> > Something I mentioned four months ago.  Alan fixed the only place in BFD
> > that I know of which assumed the size of this structure, but the extra
> > packed attribute is a guard against future problems.
> 
> I don't really like this.  A feature we try to preserve in the
> binutils is for them to serve to bootstrap the GNU tools on a new
> system.  Adding a gcc specific feature like that to elf/external.h can
> potentially hide a problem which would arise when bootstrapping with a
> non-gcc compiler.  I would prefer to leave the problem exposed, so
> that it is more likely to be fixed before it becomes a problem.

If you look at the prior art (Versym, some lines away in the same
file), and the previous discussion, a number of people seem to
disagree.

I don't care either way.  The bug I found is fixed.  Further bugs of
this nature will show up on arm-linux native and basically nowhere
else, so they will probably never be fixed.

-- 
Daniel Jacobowitz
MontaVista Software                         Debian GNU/Linux Developer