RFR/RFC: C2: limited support for vbox/vunbox
Tobias Hartmann
tobias.hartmann at oracle.com
Tue Jan 3 08:35:16 UTC 2017
Hi Zoltan,
your changes look good to me!
Here are some minor suggestions (you don't need to send another webrev):
- in line 150 of ValueTypeTestBench.java you could use rI and rL to randomize the argument values
- ValueCapableClass1::hash() is not used
- ValueCapableClass1.java is missing the copyright header
Best regards,
Tobias
On 20.12.2016 15:33, Zoltán Majó wrote:
> Hi,
>
>
> this patch adds basic C2 support for the vbox/vunbox bytecodes.
>
> http://cr.openjdk.java.net/~zmajo/valhalla/02.vbox-vunbox.webrev.00/
>
> The suggested change is in a prototype phase, thus it works for only a limited set of use cases (covered by tests 37--40). In more detail, the patch
> - does not enable C2 to generate checks currently performed by the interpreter to verify that the bytecodes handle a value-capable class and it's corresponding derived value type;
> - relies on the assumption that the memory layout of the derived value type is the same as the memory layout of the value-capable class.
>
> The patch also contains a minor refactoring of method names in ValueTypeNode so that the names better match with the HotSpot Style Guide.
>
> I tested with ValueTypeTestBench.java and with the Octane+Nashorn benchmarks.
>
> Thank you!
>
> Best regards,
>
>
> Zoltan
>
More information about the valhalla-dev
mailing list