On Tue, 24 Sep 2024 20:21:48 GMT, Chen Liang <[email protected]> wrote:
> This implementation code was written in JDK 7, before storeFence was > available in JDK 8. We should update this legacy code to make it clear. I have been harboring doubts `syncAll` even works (`FIXME: NYI` is fun) with a lone barrier for about 10-ish years now. But it is irrelevant for the patch at hand, and kicking out excess `AtomicInteger` is good. ------------- Marked as reviewed by shade (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/21169#pullrequestreview-2328938024
