> `ZoneOffset` instances are cached by the `ZoneOffset` class itself for values > in the range [-18h, 18h] for each second that is on an even quarter of an > hour (i.e. at most 2*18*4+1 = 145 values). > > Instead of using a `ConcurrentHashMap` for caching instanced, we could > instead use an `AtomicReferenceArray` with direct slot value access for said > even seconds. This will improve performance and reduce the number of object > even though the backing array will go from an initial 32 in the CHM to an > initial/final 145 in the ARA. The CHM will contain much more objects and > array slots for typical numbers of entries in the cache and will compute > hash/bucket/collision on the hot code path for each cache access.
Per Minborg has updated the pull request incrementally with three additional commits since the last revision: - Remove unused setup method - Rename method in test - Add copyright header ------------- Changes: - all: https://git.openjdk.org/jdk/pull/12346/files - new: https://git.openjdk.org/jdk/pull/12346/files/b89a9aae..29674b14 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=12346&range=07 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=12346&range=06-07 Stats: 32 lines in 3 files changed: 23 ins; 8 del; 1 mod Patch: https://git.openjdk.org/jdk/pull/12346.diff Fetch: git fetch https://git.openjdk.org/jdk pull/12346/head:pull/12346 PR: https://git.openjdk.org/jdk/pull/12346
