RFR: 8292458: Atomic operations on scoped enums don't build with clang [v2]
Erik Österlund
eosterlund at openjdk.org
Wed Aug 17 05:26:26 UTC 2022
On Wed, 17 Aug 2022 02:05:18 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:
>> Please review this change to allow the Atomic class to support scoped enum
>> values on all platforms. We do this by adding PrimitiveConversions::Translate
>> support for enums (converting to/from the underlying type), and changing
>> relevant Atomic operations to use that translation instead of trying to handle
>> enums on the integer code path.
>>
>> Testing:
>> mach5 tier1-3, including gtests added by this change
>
> Kim Barrett has updated the pull request incrementally with one additional commit since the last revision:
>
> use _t suffixed type traits
Looks good. Thank you for fixing this.
-------------
Marked as reviewed by eosterlund (Reviewer).
PR: https://git.openjdk.org/jdk/pull/9895
More information about the hotspot-runtime-dev
mailing list