RFR: 8367717: Cleanup atomic_copy64 [v2]
Justin King
jcking at openjdk.org
Mon Oct 6 14:26:16 UTC 2025
On Mon, 6 Oct 2025 09:38:24 GMT, jonghoonpark <duke at openjdk.org> wrote:
>> related jira issue: https://bugs.openjdk.org/browse/JDK-8367717
>>
>> ## Changes
>> - deleted all definitions of atomic_copy64
>> - replaced all calls to `atomic_copy64` with `AtomicAccess::store(dst, AtomicAccess::load(src))`
>>
>> ## Verification
>> - Verified by running:
>> - `make run-test TEST="gtest:StubRoutines*"`
>> - `make run-test TEST="gtest:AtomicAccess*"`
>>
>> Please let me know if there are any additional or more appropriate verification steps I should perform.
>
> jonghoonpark has updated the pull request incrementally with one additional commit since the last revision:
>
> cleanup: align to use AtomicAccess
>
> Signed-off-by: jonghoonpark <dev at jonghoonpark.com>
This partially fixes https://bugs.openjdk.org/browse/JDK-8369207 as well, but this only does it for `jlong`. I can wait for this to go in, and then update `jshort` and `jint` to use `AtomicAccess` as well or we can do it all in this PR.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/27642#issuecomment-3371948431
More information about the hotspot-runtime-dev
mailing list