RFR: 8230203: Replace markWord enums with uintptr_t constants
Doerr, Martin
martin.doerr at sap.com
Wed Aug 28 08:02:42 UTC 2019
Hi Kim,
thanks for your explanations.
> What I think you want is
> C++11 explicitly typed enums, e.g. “enum : uintptr_t { … };”
Yes. That's what I meant.
> I agree there’s a lot less boilerplate with that. But I like all the cast removals
> coming out of this change, and would rather not wait for those.
I also like the cast removals.
> There are still some issues with Solaris Studio. Potential issues with other
> (more obscure) platforms.
I hope they can get resolved soon.
So wrt. to this issue, I abstain.
Cast removal looks good, tons of static consts look bad.
Best regards,
Martin
More information about the hotspot-dev
mailing list