RFR: 8346567: Make Class.getModifiers() non-native [v2]
Coleen Phillimore
coleenp at openjdk.org
Thu Feb 6 12:11:14 UTC 2025
On Wed, 5 Feb 2025 21:44:37 GMT, Dean Long <dlong at openjdk.org> wrote:
>> @DanHeidinga Great explanation, thank you!
>
> If Class had other fields smaller than `int`, would be consider making this something like `char` to save space (allowing all the sub-word fields to be compacted)?
I thought of doing this since I made modifiers u2 in the Hotspot code just previously, but all the Java code refers to this as an int. And I didn't see other fields to compact it with. Maybe if access_flags are moved we could make them both char (not short since they're unsigned). It feels weird to not have unsigned short to my C++ eyes.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22652#discussion_r1944613105
More information about the serviceability-dev
mailing list