Request for reviews (S): 7048332: Cadd_cmpLTMask doesn't handle 64-bit tmp register properly

Vladimir Kozlov vladimir.kozlov at oracle.com
Wed May 25 19:52:52 PDT 2011


Thank you, Tom

Vladimir

Tom Rodriguez wrote:
> Looks good.
> 
> tom
> 
> On May 25, 2011, at 7:46 PM, Vladimir Kozlov wrote:
> 
>> http://cr.openjdk.java.net/~kvn/7048332/webrev
>>
>> Fixed 7048332: Cadd_cmpLTMask doesn't handle 64-bit tmp register properly
>>
>> enc_cmpLTP_mem() code relied on tmp be bound to rcx register but after 6511991 changes it was not true any more. Use ins_encode %{ %} form to encode cadd_cmpLTMask() instruction and remove unused code.
>>
>> Added regression test.
> 


More information about the hotspot-compiler-dev mailing list