On Wed, 26 Jul 2023 12:20:49 GMT, David Holmes <dhol...@openjdk.org> wrote:
>> test/hotspot/jtreg/serviceability/jvmti/vthread/GetThreadState/GetThreadStateTest.java
>> line 133:
>>
>>> 131: } finally {
>>> 132: thread.join();
>>> 133: Reference.reachabilityFence(lock);
>>
>> It would be nice to add a short comment why this is needed.
>
> I can't see why it would be needed.
> It would be nice to add a short comment why this is needed.
It's not needed here, it was needed in another version of the tests. While
benign, I can remove it to avoid any confusion. Thanks.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14978#discussion_r1275293060
