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