RFR: 8184017: [MVT] Support Q-types in MHs.insertArguments()

Paul Sandoz paul.sandoz at oracle.com
Mon Jul 10 19:28:16 UTC 2017


> On 10 Jul 2017, at 06:06, Vladimir Ivanov <vladimir.x.ivanov at oracle.com> wrote:
> 
> Thanks, Maurizio & Paul.
> 

+1 on test.

Paul.

> Yes, all tests in jdk/test/valhalla/mvt pass.
> 
> I decided to write a dedicated unit test and here's an updated version:
>  http://cr.openjdk.java.net/~vlivanov/valhalla/8184017/webrev.01/
> 
> Also, added explicit null check to align the behavior on MVTs with primitives.
> 
> Best regards,
> Vladimir Ivanov




More information about the valhalla-dev mailing list