RFR: 8292458: Atomic operations on scoped enums don't build with clang [v2]

Stefan Karlsson stefank at openjdk.org
Wed Aug 17 07:10:53 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. Thanks for fixing.

-------------

Marked as reviewed by stefank (Reviewer).

PR: https://git.openjdk.org/jdk/pull/9895


More information about the hotspot-runtime-dev mailing list