Request for review(XS): 7005241: C1: SEGV in java.util.concurrent.LinkedTransferQueue.xfer() with compressed oops

[Vladimir Kozlov]

Good.

Vladimir

Igor Veresov wrote:
> On 12/7/10 5:20 PM, Vladimir Kozlov wrote:
>> Looks good. As discussed add a comment that cmpval(rax) is used
>> implicitly in cmpxchg instruction.
>>
> 
> Done. Thanks!
> 
> igor
> 
>> Thanks,
>> Vladimir
>>
>> Igor Veresov wrote:
>>> Implementation of the CAS primitive for x64 compressed oops was
>>> incorrect. It kills rscratch2 register (r11), which is allocatable in
>>> C1. Also, we don't need to restore cmpval as it's never used after
>>> that, so we need only one temporary register, which can be scratch1.
>>>
>>> Webrev: http://cr.openjdk.java.net/~iveresov/7005241/webrev.00
>>>
>>> Tested with the failed nightly test.
>>>
>>> Thanks,
>>> igor
>>>
>