On Tue, 21 Oct 2025 13:11:45 GMT, Fredrik Bredberg <[email protected]> wrote:
> This is the last PR in a series of PRs (see: > [JDK-8344261](https://bugs.openjdk.org/browse/JDK-8344261)) to obsolete the > LockingMode flag and related code. > > The main focus is to to unify `ObjectSynchronizer` and > `LightweightSynchronizer`. > There used to be a number of "dispatch functions" to redirect calls depending > on the setting of the `LockingMode` flag. > Since we now only have lightweight locking, there is no longer any need for > those dispatch functions, so I removed them. > To remove the dispatch functions I renamed the corresponding lightweight > functions and call them directly. > This ultimately led me to remove "lightweight" from the function names and go > back to "fast" instead, just to avoid having some with, and some without the > "lightweight" part of the name. > > This PR also include a small simplification of > `ObjectSynchronizer::FastHashCode`. > > Tested tier1-7 (on supported platforms) without seeing any problems that can > be traced to this code change. > All other platforms (`arm`, `ppc`, `riscv`, `s390`) has been sanity checked > using QEMU. @bulasevich, @TheRealMDoerr, @RealFYang, @offamitkumar Hi guys! This is the last PR in a series of PRs to obsolete the `LockingMode` flag and related code. So please please grab a copy, run it on your favorite platform, and tell me if there's anything wrong with it. Thanks in advance. ------------- PR Comment: https://git.openjdk.org/jdk/pull/27915#issuecomment-3435760553
