RFR: 8308655: Narrow types of ConstantPool and ConstMethod returns
    Coleen Phillimore 
    coleenp at openjdk.org
       
    Tue May 23 15:30:36 UTC 2023
    
    
  
This change uses a number of ways to eliminate -Wconversion warnings in the metadata files in the oops directory. 
1. narrow return types to u2 if the accessor is for a field or value that's u2 (u2 is most common for constMethod fields and constant pool indices)
2. Use checked_cast<type> for places where we know the int value is u2 or s2 but propagating these types is too much fan out.
3. Use plain casts where it's obvious that the int value fits in the casted-to type.
4. Moved KlassKind to be contained in Klass to add the Unknown enum value to use instead of -1.
5. Moved the compute_from_signature function into ConstMethod as it sets values in ConstMethod and the parameters are changed in the set functions.  Removed some pass through functions in Method.
Tested with tier1-4.
-------------
Commit messages:
 - merge error
 - Fix ConstantPool types.
 - Refine ConstMethod parameter types and int size checking.
Changes: https://git.openjdk.org/jdk/pull/14092/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=14092&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8308655
  Stats: 201 lines in 31 files changed: 43 ins; 42 del; 116 mod
  Patch: https://git.openjdk.org/jdk/pull/14092.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/14092/head:pull/14092
PR: https://git.openjdk.org/jdk/pull/14092
    
    
More information about the serviceability-dev
mailing list