Request for reviews (M): 7059034: Use movxtod/movdtox on T4

Vladimir Kozlov vladimir.kozlov at oracle.com
Thu Jul 7 09:37:18 PDT 2011


Thank you, Tom

Vladimir

Tom Rodriguez wrote:
> These look good.
> 
> tom
> 
> On Jul 6, 2011, at 4:04 PM, Vladimir Kozlov wrote:
> 
>> http://cr.openjdk.java.net/~kvn/7059034/webrev
>>
>> Fixed 7059034: Use movxtod/movdtox on T4
>>
>> Currently VM uses stack to move data between general and float registers on Sparc. Use new VIS3 mov instructions on T4 for such moves.
>>
>> I removed R_F30,R_F31 from dflt_low_reg since they used as temps (see line 412).
>> Also fixed float<->integer conversion instructions since they used incorrect Width for float register encoding. For example, encoding failed for next instr ftoi(FloatRegisterImpl::D,F4,F1) and fitof(FloatRegisterImpl::D,F1,F8).
>>
>> Tested with CTW, nsk.
>> I attached microbenchmark test to the bug report.
> 


More information about the hotspot-compiler-dev mailing list