This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC 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] include dbxelf.h via tm_file rather than thru elfos.h.


 > 2001-12-11  David O'Brien  <obrien@FreeBSD.org>
 > 
 >         * config.gcc (m68020-*-elf*): Expand tm_file to include all needed
 >         cpu configuration headers.
 >         * config/h8300/elf.h: Include dbxelf.h.
 >         * config/m68k/linux.h: Likewise.
 >         * config/m68k/m68020-elf.h: Likewise.
 >         * config/m68k/m68kv4.h: Likewise.

David,

I think this patch of yours broke "m68k-*-rtemself*|m68k-*-rtems*".
(I tried a cross to m68k-unknown-rtemself.)  That port uses
m68k/rtemself.h which includes m68k/m68020-elf.h.  This hunk:

 > diff -u -r1.4 m68020-elf.h
 > --- m68020-elf.h	1999/12/14 23:14:42	1.4
 > +++ m68020-elf.h	2001/12/10 23:29:39
 > @@ -21,12 +21,6 @@
 >  
 >  /* This comment is here to see if it will keep Sun's cpp from dying.
 >  */
 >  
 > -#include "m68k/m68k-none.h"
 > -#include "m68k/m68kelf.h"
 > -#include "elfos.h"
 > -/* m68k/m68kemb.h sets some macros in ways that override the svr4
 > -#abi.  */

wasn't reflected in the tm_file for m68k-rtemself.  I'm not sure you
can simply add these files to config.gcc because m68k/rtemself.h
defines MOTOROLA before including anything and I suspect it has an
impact which would be lost in the move to tm_file.

Would you please fix it?

		Thanks,
		--Kaveh
--
Kaveh R. Ghazi			Engagement Manager / Project Services
ghazi@caip.rutgers.edu		Qwest Internet Solutions


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