Request for reviews (XL): 6919934: JSR 292 needs to support x86 C1

Tom Rodriguez Thomas.Rodriguez at Sun.COM
Tue Mar 9 11:25:35 PST 2010


Looks good.

tom

On Mar 9, 2010, at 3:33 AM, Christian Thalinger wrote:

> On Tue, 2010-02-16 at 12:05 -0800, Tom Rodriguez wrote:
>>> I can change it yes, but I'm really not sure yet if we need to preserve
>>> the SP on SPARC.
>> 
>> I don't think they are and I'd rather not have XXX comments lying
>> around.  If they are needed then I suspect you'll figure it out
>> because of a new bug report, not because the comment is there.
> 
> Since I've already started the SPARC C1 port I know that we need to
> implement these methods.  I removed the comment and put an
> Unimplemented() there instead.
> 
>>> No, RSI is an argument register on 64 bit and I didn't know which one to
>>> use instead.  I guess any callee saved might do it?
>> 
>> Yep.
> 
> I used R14.
> 
> Here is an updated webrev and I will push that one:
> 
> http://cr.openjdk.java.net/~twisti/6919934/webrev.02/
> 
> -- Christian
> 



More information about the hotspot-compiler-dev mailing list