RFR: 8372564: Convert StringDedup to use Atomic<T> [v2]
Thomas Schatzl
tschatzl at openjdk.org
Fri Dec 19 10:06:26 UTC 2025
On Mon, 15 Dec 2025 01:42:40 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:
>> Please review this change to StringDedup to use Atomic rather than directly
>> applying AtomicAccess to volatile members.
>>
>> Testing: mach5 tier1-5
>
> Kim Barrett has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
>
> - Merge branch 'master' into stringdedup-atomic
> - remove use of DeferredStatic
> - Merge branch 'master' into stringdedup-atomic
> - stringdedup uses Atomic
Still good after the merge to me.
-------------
Marked as reviewed by tschatzl (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/28524#pullrequestreview-3597896648
More information about the hotspot-gc-dev
mailing list