Re: [patch] Add start of mips32/mips64 support to sim/mips/mips.igen

[cgd at sibyte dot com (Chris G. Demetriou)]

[ To reply to two messages in one ]

> To expand on this one.  I've two concerns.
> 
> 	o	I'd like to see a patch tabled
> 		that adds the full mips32/mips64 ISA.
> 
> 		I think a new ISA should be added
> 		as a single patch and not as an
> 		incremental set of changes.

OK.  Having worked in the other tools, my inclination was stepwise
refinement.

We can certainly do this if you are so inclined.  Would you prefer:

(a) 1 patch which adds MIPS V, MIPS32, and MIPS64 at once, or

(b) 1 patch for each.

I think if the goal is to do "one patch that just does it," I'd prefer
the former, because it means a lot less duplication of effort.


> 	o	I'd like to be sure that the information
> 		being committed is, one way, or another
> 		public.
> 
> 		Instruction set simulators include a semantic
> 		specification of each instruction.  I'd
> 		like to be sure that your allowed to publish/
> 		submit this for the ISS.		
> 
> If there is published documentation that you can refer people to then
> things are pretty easy.  If there isn't then I'll work on the assumption
> that you've gone through due process on this.

There is some public documentation on MIPS32, as part of the MIPS32 4K
core documentation available on:

	http://www.mips.com/publications/index.html

For the rest, yes, we've done the due process.  8-)


> "Chris G. Demetriou" wrote:
> [there was an interveening e-mail]
>
> > Andrew Cagney <ac131313@cygnus.com> writes:
> > > Two quick questions first off.
> > >
> > >       o       what is mipsV?
>
> Sorry, what I ment to say was that the change adds ``mipsV'' yet nothing
> else explains this.  If nothing else it is most likely a separate change.

Oh, I'm sorry.  I thought i'd mentioned it in my ChangLog, along with
the mips32 and mips64 entries, but I guess I didn't.

It's another ISA, like MIPS I - MIPS IV.  (It incorporates all of MIPS
IV and adds on to it.)  If i recall correctly, its main addition is
Paired Single floating point operations, but it's been a while since
I've checked.



chris