On Thu, 9 Mar 2023 23:17:39 GMT, Daniel D. Daugherty <dcu...@openjdk.org> wrote:
>> Roman Kennke has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 99 commits: >> >> - Merge branch 'master' into JDK-8291555-v2 >> - Various small fixes and improvements >> - Merge remote-tracking branch 'origin/JDK-8291555-v2' into JDK-8291555-v2 >> - Use realloc instead of malloc+copy when growing the lock-stack >> - Inline initial LockStack stack >> - Fix interpreter asymmetric fast-locking >> - Fix merge error (move done label into correct places) >> - Merge branch 'master' into JDK-8291555-v2 >> - Small fixes >> - Fix anon owner in fast-path, avoid runtime call (aarch64) >> - ... and 89 more: https://git.openjdk.org/jdk/compare/5726d31e...f9f93b36 > > src/hotspot/cpu/arm/c1_MacroAssembler_arm.cpp line 56: > >> 54: } >> 55: >> 56: void C1_MacroAssembler::build_frame(int frame_size_in_bytes, int >> bang_size_in_bytes, int max_monitors) { > > So the `max_monitors` param is added, but not use of it. > Is someone else doing the 32-bit ARM port? Hopefully :-) I currently can't do it, though. With fast-locking (and the rest of Lilliput) behind an experimental flag, this is probably ok for now? ------------- PR: https://git.openjdk.org/jdk/pull/10907
