On Fri, 12 Jul 2024 15:56:59 GMT, Roman Kennke <[email protected]> wrote:
>> Axel Boldt-Christmas has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Update arguments.cpp
>
> src/hotspot/share/runtime/synchronizer.cpp line 390:
>
>> 388:
>> 389: static bool useHeavyMonitors() {
>> 390: #if defined(X86) || defined(AARCH64) || defined(PPC64) ||
>> defined(RISCV64) || defined(S390)
>
> Why are those if-defs here? Why is ARM excluded?
Oh I see, you only moved this up. Still a bit puzzling.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20067#discussion_r1676142470
