On Mon, 17 Apr 2023 20:00:58 GMT, Roman Kennke <rken...@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 156 commits: >> >> - Merge remote-tracking branch 'upstream/master' into JDK-8291555-v2 >> - A few more LM_ prefixes in 32bit code >> - Replace UseHeavyMonitor with LockingMode == LM_MONITOR >> - Prefix LockingMode constants with LM_* >> - Bunch of comments and typos >> - Don't use NativeAccess in LockStack::contains() >> - RISCV update >> - Put back thread type check in OS::is_lock_owned() >> - Named constants for LockingMode >> - Address David's review comments >> - ... and 146 more: https://git.openjdk.org/jdk/compare/d2ce04bb...d0a448c6 > > Hi there, > what is needed to bring this PR over the approval line? @rkennke - I'm planning to do another crawl thru review next week. ------------- PR Comment: https://git.openjdk.org/jdk/pull/10907#issuecomment-1518372821