RFR: 8316961: Fallback implementations for 64-bit Atomic::{add,xchg} on 32-bit platforms [v4]
Aleksey Shipilev
shade at openjdk.org
Mon Oct 23 14:29:42 UTC 2023
On Mon, 23 Oct 2023 14:17:58 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
>> See the bug for rationale. Looks like there is enough infrastructure to achieve what we want without significant fan-out. I checked all `atomic_*.hpp` headers for unimplemented `PlatformAdd<8>` and `PlatformXchg<8>`, and only these seem to be affected.
>>
>> Unfortunately, we cannot test these apart from the existing gtest.
>>
>> Additional testing:
>> - [x] linux-x86-server-fastdebug, atomic tests pass
>> - [x] linux-arm-server-fastdebug, atomic tests pass (with #16269 applied)
>
> Aleksey Shipilev has updated the pull request incrementally with one additional commit since the last revision:
>
> Use public methods instead of Platform*
> Yes, and Atomic::load instead of PlatformLoad. So actually follow the model of the atomic bitset functions.
Right. Done in new commit. The previous version was from the time I did these fallbacks straight in platform atomic files. But in the shared code, we should definitely go for the non-platform methods. I am re-running tests, but expect no trouble.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/16252#issuecomment-1775329068
More information about the hotspot-dev
mailing list