On Wed, 27 Aug 2025 19:40:28 GMT, Roger Riggs <[email protected]> wrote:
> As observed in [JDK-8366043](https://bugs.openjdk.org/browse/JDK-8366043) > [lworld] (LIFE = Legacy Idiom For Equality) causes performance regressions. > Updating HashMap and ConcurrentHashMap to use `java.util.Objects.equals` will > make it easier to measure performance of options that remove or modify the > use of `==` > > Replace constructs like: > > - ((k = e.key) == key || (key != null && > key.equals(k)))) > with: > + Objects.equals(key, k)) > > > The changes in ConcurrentHashMap are a bit different due to the use of null > as a sentinel. > > The order of arguments to the .equals methods must remain the same to ensure > compatibility. This pull request has now been integrated. Changeset: c8d4a247 Author: Roger Riggs <[email protected]> URL: https://git.openjdk.org/valhalla/commit/c8d4a247861052aa6ed43125bcbe49995326938f Stats: 45 lines in 2 files changed: 1 ins; 7 del; 37 mod 8366214: [lworld] Use Objects.equals in HashMap and ConcurrentHashmap Reviewed-by: liach ------------- PR: https://git.openjdk.org/valhalla/pull/1536
