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