On Wed, 29 Apr 2026 00:23:28 GMT, Chris Plummer <[email protected]> wrote:

>> Serguei Spitsyn has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   review: a call to OrderAccess::storestore is added
>
> src/hotspot/share/prims/jvmtiImpl.cpp line 503:
> 
>> 501:     // Must ensure the content of the buffered value is visible
>> 502:     // before publishing the buffered value oop
>> 503:     OrderAccess::storestore();  }
> 
> Suggestion:
> 
>     OrderAccess::storestore();
>   }

Addressed, thanks.

-------------

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2363#discussion_r3158384487

Reply via email to