RFR: 8320318: ObjectMonitor Responsible thread [v3]
Fredrik Bredberg
fbredberg at openjdk.org
Wed Sep 25 14:18:51 UTC 2024
On Fri, 20 Sep 2024 04:55:16 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> Fredrik Bredberg has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Update two, after the review
>
> src/hotspot/cpu/aarch64/c2_MacroAssembler_aarch64.cpp line 535:
>
>> 533:
>> 534: // Set owner to null.
>> 535: // Release to satisfy the JMM
>
> Can I suggest you use this comment form in each of the cpu-specific files. At the moment code that does the same thing is commented slightly differently with regard to the need for a release-store. Thanks.
fixed
> src/hotspot/cpu/x86/c2_MacroAssembler_x86.cpp line 499:
>
>> 497: #endif
>> 498:
>> 499: // Intentional fall-through into slow path
>
> I don't think this comment makes sense / applies any more.
fixed
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19454#discussion_r1775314987
PR Review Comment: https://git.openjdk.org/jdk/pull/19454#discussion_r1775315614
More information about the hotspot-dev
mailing list