RFR: 8367486: Change prefix for platform-dependent AtomicAccess files [v4]
Kim Barrett
kbarrett at openjdk.org
Mon Sep 15 23:46:10 UTC 2025
On Mon, 15 Sep 2025 06:19:20 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:
>> JDK-8367014 renamed the class Atomic to AtomicAccess and renamed atomic.hpp to atomicAccess.hpp. I propose that we also rename the platform-dependent headers that implement the AtomicAccess support to be named atomicAccess_<os>_<arch>.hpp.
>>
>> Note that I left atomic_aarch64.hpp and atomic_linux_aarch64.S intact since they clear, external usages outside of the AtomicAccess class.
>>
>> I will verify that this still builds on the platform that I can easily build.
>
> Stefan Karlsson has updated the pull request incrementally with two additional commits since the last revision:
>
> - Update include in os_bsd_zero.cpp
> - Update include in os_linux_zero.cpp
Looks good.
-------------
Marked as reviewed by kbarrett (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/27245#pullrequestreview-3226621479
More information about the hotspot-runtime-dev
mailing list