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