Subject: Re: proposed change to m68k-gcc CPP_SPEC / CPP_PREDEFINES
To: Jason Thorpe <thorpej@nas.nasa.gov>
From: J.T. Conklin <jtc@cygnus.com>
List: tech-userlevel
Date: 05/07/1996 16:55:54
>> The enclosed patchq removes mc68000 and mc68020 from our gcc's
>> CPP_PREDEFINES, and defines __mc680X0__ as appropriate for 
>> whatever -m680X0 flag is specified on the command line.
>> 
>> This change is consistant with the goal of having only __NetBSD__ and
>> __${MACHINE_ARCH}__ be predefined CPP macros, yet it allows users to
>> have conditional code based on the target cpu.

> ...but, __${MACHINE_ARCH}__ is __m68k__ ... I don't mind seeing 
> __mc680X0__ as well (as optional, i.e. iff you specify -m680X0 on the 
> command line), but __m68k__ is more inline with the goal.

Take a look at the patch again.  Note that I didn't remove the -Dm68k
from CPP_PREDEFINES.  Thus __m68k__ will continue to be defined (thx
to gcc magic).  I consider changing CPP_PREDEFINES so only __m68k__
(and not m68k and __m68k) is defined a separate issue.

	--jtc