[lworld] RFR: 8295234: [lworld] inner class of older classfile versions is not recognized as identity class
    Frederic Parain 
    fparain at openjdk.org
       
    Fri Oct 14 17:32:27 UTC 2022
    
    
  
On Fri, 14 Oct 2022 00:10:57 GMT, Mandy Chung <mchung at openjdk.org> wrote:
> The modifiers of an inner class uses the access flags in the `InnerClasses`  attribute.   An inner class with an older class file version does not have `ACC_IDENTITY` flag set.   Instead of having `Class::isIdentity`  to depend on `getModifiers()`, this PR proposes to make `Class::isIdentity` a native method implemented by the VM. In addition, `Class::isIdentity` can be made as intrinsics.
> 
> I leave `Class::isValue` as is.   A value class must have `ACC_VALUE` flag set.   `Class::getModifiers` is already intrinsified.   We can revisit this in the future when there is a need to make it as a native VM method.
Looks good to me.
Fred
-------------
Marked as reviewed by fparain (Committer).
PR: https://git.openjdk.org/valhalla/pull/794
    
    
More information about the valhalla-dev
mailing list