This is the mail archive of the
gcc-bugs@gcc.gnu.org
mailing list for the GCC project.
Re: Changes for a `m68k-motorola-sysv'
- To: Manfred Hollstein <manfred at s-direktnet dot de>
- Subject: Re: Changes for a `m68k-motorola-sysv'
- From: Jeffrey A Law <law at hurl dot cygnus dot com>
- Date: Fri, 05 Sep 1997 14:15:34 -0600
- cc: egcs-bugs at cygnus dot com
- Reply-To: law at cygnus dot com
In message <199708311443.QAA09628@saturn.s-direktnet.de>you write:
> These changes are necessary for a clean build and test run if
> configured for a `m68k-motorola-sysv' machine (aka `delta'). I already
> tried several times to convince R. Kenner to adopt the TARGET_SWITCHES
> stuff but had no luck so far :-( Hopefully, I'll be more successfull
> here :-).
Some general notes.
Send independent patches separately; doing so make it much easier for us
to go ahead and install those which are obviously correct without holding
up patches that need further thought and/or discussion.
Second, describe what problem you're trying to solve with each patch;
"build on m68k-motorola-sysv" really isn't much help because most of
us don't have one or know anything about it.
Don't send diffs to the ChangeLog -- just send the entries as raw
text. Diffs to the ChangeLog rarely apply cleanly.
> + * sdbout.c (plain_type_1, case ARRAY_TYPE): From Jim Wilson:
> + Verify that TYPE_DOMAIN has integer TYPE_{MAX,MIN}_VALUE before
> + using them.
This patch has been installed.
> + * config/m68k/m68k.h (TARGET_SWITCHES): MASK_68881 added to
> + 68000 and c68000 as no bare 68000 CPU has a 68881 FPU by default.
This seems wrong. It sounds more like the multilib support in t-mot3300
has a slight problem and needs to be fixed.
> + * config/m68k/mot3300.h (MACHINE_STATE_SAVE): Use % as prefix for
> + register names #ifdef USE_GAS, as GNU as needs it.
> + (MACHINE_STATE_RESTORE): Ditto.
Is there any way to merge this into the main MACHINE_{SAVE,RESTORE}_STATE
in m68k.h?
Jeff