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