RFR(S) 8191437: AOT doesn't work easily after thread local handshakes

[Vladimir Kozlov]

On 11/28/17 4:49 PM, dean.long at oracle.com wrote:
> On 11/28/17 4:12 PM, Vladimir Kozlov wrote:
> 
>> On 11/28/17 10:00 AM, dean.long at oracle.com wrote:
>>> https://bugs.openjdk.java.net/browse/JDK-8191437
>>>
>>> http://cr.openjdk.java.net/~dlong/8191437/webrev.hs/
>>
>> VM changes looks fine to me.
>>
> 
> Thanks Vladimir.
> 
>>>
>>> http://cr.openjdk.java.net/~dlong/8191437/webrev.graal/
>>>
>>>
>>> This change adds support for thread local handshakes to Graal for x64 and SPARC.  Graal generates a different 
>>> instruction encoding on x64 than C1/C2, so I had to account for that difference in
>>
>> I assume it works in all cases now (C1, C2, Graal generated code) ?
> 
> Yes.
> 
>>
>>> sharedRuntime_x86_64.cpp.  I'm including the Graal changes because I would like to push them together with the 
>>> HotSpot changes.
>>
>> I would still suggest to do review and testing in Labs.
>>
> 
> Yes, I waited until it was reviewed and pushed upstream to Graal before sending out this RFR.

Good.

Thanks,
Vladimir

> 
> dl
> 
>> Thanks,
>> Vladimir
>>
>>>
>>>
>>> dl
>>>
>