RFR: 8177397: [JVMCI] remove unnecessary synchronization

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

RFR: 8177397: [JVMCI] remove unnecessary synchronization

Doug Simon @ Oracle
Please review this tiny change to remove some unnecessary synchronization.

https://bugs.openjdk.java.net/browse/JDK-8177397
http://cr.openjdk.java.net/~dnsimon/8177397/

-Doug
Reply | Threaded
Open this post in threaded view
|

Re: RFR: 8177397: [JVMCI] remove unnecessary synchronization

Tobias Hartmann-2
Hi Doug,

looks good to me.

Best regards,
Tobias

On 03.11.2017 15:55, Doug Simon wrote:
> Please review this tiny change to remove some unnecessary synchronization.
>
> https://bugs.openjdk.java.net/browse/JDK-8177397
> http://cr.openjdk.java.net/~dnsimon/8177397/
>
> -Doug
>
Reply | Threaded
Open this post in threaded view
|

Re: RFR: 8177397: [JVMCI] remove unnecessary synchronization

Doug Simon @ Oracle

> On 3 Nov 2017, at 16:02, Tobias Hartmann <[hidden email]> wrote:
>
> Hi Doug,
>
> looks good to me.

Thanks for the review.

-Doug

>
> Best regards,
> Tobias
>
> On 03.11.2017 15:55, Doug Simon wrote:
>> Please review this tiny change to remove some unnecessary synchronization.
>>
>> https://bugs.openjdk.java.net/browse/JDK-8177397
>> http://cr.openjdk.java.net/~dnsimon/8177397/
>>
>> -Doug
>>

Reply | Threaded
Open this post in threaded view
|

Re: RFR: 8177397: [JVMCI] remove unnecessary synchronization

Vladimir Kozlov
In reply to this post by Tobias Hartmann-2
+1

Vladimir

On 11/3/17 8:02 AM, Tobias Hartmann wrote:

> Hi Doug,
>
> looks good to me.
>
> Best regards,
> Tobias
>
> On 03.11.2017 15:55, Doug Simon wrote:
>> Please review this tiny change to remove some unnecessary synchronization.
>>
>> https://bugs.openjdk.java.net/browse/JDK-8177397
>> http://cr.openjdk.java.net/~dnsimon/8177397/
>>
>> -Doug
>>