XCOFF64 bfd changes

Ian Lance Taylor ian@zembu.com
Fri Apr 7 18:31:00 GMT 2000


   Date: Fri, 7 Apr 2000 16:54:25 -0500
   From: Clinton Popetz <cpopetz@cygnus.com>

   *** bfd/coff-rs6000-64.c	Wed Dec 31 18:00:00 1969
   --- bfd/coff-rs6000-64.c	Wed Mar 29 13:50:20 2000

Please use a different name which is unique in 8.3.  Run doschk to
check.  We don't worry about it too much, but we might as well avoid
adding yet another problem.

   + #include "coff-rs6000.c"

I don't like this approach.  I know that BFD uses it a lot, for
historical reasons.  But this is another thing I'd rather we tried to
avoid in new code.

The way to do this is to use backend functions.  I happen to like the
ECOFF support, since I wrote pretty much all of it.  We support two
different kinds of ECOFF in different sizes without any bizarre file
inclusion.

   *** include/coff/rs6000.h	1999/05/03 07:29:03	1.1.1.1
   --- include/coff/rs6000.h	2000/04/07 21:32:24

Adding ifdefs in this file is also problematical.  It becomes harder
to understand how to use the files in include/coff.  These files are
sometimes included by machine independent files.  Most of the code in
include/coff avoids ifdefs, with the exception of the PE support.

Neither of these two issues are show-stoppers for this patch.  But I
strongly encourage you to discuss a possible design on the binutils
list before implementing it.

Ian


More information about the Binutils mailing list