RFR (s): 8208162: Support for multi-dimensional value arrays is incomplete
    Karen Kinnear 
    karen.kinnear at oracle.com
       
    Wed Jul 25 13:15:50 UTC 2018
    
    
  
Mr Simms,
Code changes look good and logic makes sense.
thanks for the quick fix!
Karen
> On Jul 25, 2018, at 6:16 AM, David Simms <david.simms at oracle.com> wrote:
> 
> Greetings,
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8208162
> 
> Webrev: http://cr.openjdk.java.net/~dsimms/valhalla/8208162/
> 
> I believe the intention of the "ObjArrayKlass::_bottom_klass" field was always to hold an instance klass that contained user class that might require a number of checks, e.g. access checking for reflection, module access etc...
> 
> A value of "typeArrayKlass" simply denoted, "system type". I believe assigning a valueArrayKlass to this field to be incorrect, the fix is simply to pick out the valueArrayKlass element type as "bottom klass"
> 
> Testing: internal tier1, and Ioi's consistency tests now exhibit correct behavior.
> 
> Cheers
> 
> /David Simms
> 
> 
> 
    
    
More information about the valhalla-dev
mailing list