This is the mail archive of the cgen@sources.redhat.com mailing list for the CGEN 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: m32r port breakage


>>>>> "Doug" == Doug Evans <dje@transmeta.com> writes:

  Doug> Yes, it's been ages since I've looked at the m32r port, but
  Doug> it's the one I like to use when playing with things.
  Doug> Hans-Peter has a patch he wants to go in and before it has my
  Doug> blessing I want to play with it on a real port.

  Doug> This patch breaks the m32r simulator.

  Doug> 2001-04-02  Ben Elliston  <bje@redhat.com>
	
  Doug> 	* sim-decode.scm (@prefix@_init_idesc_table): Compute tabsize
  Doug> 	using the size of the table and its elements.
  Doug> 	(-gen-decode-insn-globals): Define the idesc table's size to be
  Doug> 	the last instruction enum plus one, not @PREFIX@_INSN_MAX.
  Doug> 	* sim-model.scm (-gen-mach-defns): Define CPU_MAX_INSNS as the
  Doug> 	last instruction enum plus one, not @CPU@_INSN_MAX.

  Doug> Why was @PREFIX@_INSN_MAX replaced with "last instruction enum plus one"?

A port I was doing had an instruction called "MAX" that generated the
symbol @PREFIX@_INSN_MAX.  Rather than change the name of the sentinel
value, I decided to clean this up to avoid any further problems.  I
guess I got carried away. :-( Perhaps you could change the sentinel
value's name to @PREFIX@_INSN__MAX?

Ben


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