This is the mail archive of the
cgen@sourceware.org
mailing list for the CGEN project.
Re: Unbreak CRIS sim: fix -gen-decode-insn-entry. And why the widened mem fetch?
- From: Hans-Peter Nilsson <hans-peter dot nilsson at axis dot com>
- To: jimb at redhat dot com
- Cc: hans-peter dot nilsson at axis dot com, cgen at sourceware dot org, brolley at redhat dot com
- Date: Fri, 2 Dec 2005 16:41:42 +0100
- Subject: Re: Unbreak CRIS sim: fix -gen-decode-insn-entry. And why the widened mem fetch?
> From: Jim Blandy <jimb@redhat.com>
> Date: Thu, 01 Dec 2005 23:06:40 -0800
> That could be due to my 2005-05-16 change. I have to admit, I didn't
> feel very confident in my understanding of that code at all, and I was
> grateful to have found a relatively straightforward way to make it at
> least produce the right field values.
Well, for all I know it did *before*, but as shown it seems
doubtful it does in all cases *after* your change. As always,
it may be that your change only uncovered another bug of course.
> If you are up to the task of
> going into utils-gen.scm and making it produce minimal and correct
> fetches, please do.
To have a fighting chance at this, I need to know what bug this
fixes. When I read the patch it seemed more of a convenience-
change. I wouldn't rule out this fixing what is actually a bug
in the .cpu file. Is there a machine description and a small
program I can use?
I guess you somehow got into trouble with a big-endian sim, but
your patch refers to this problem in terms a bit too vaguely for
me to decipher.
brgds, H-P