This is the mail archive of the gcc-bugs@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]

Re: Changes for a `m68k-motorola-sysv'


  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


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