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