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

Maurizio Cimadamore maurizio.cimadamore at oracle.com
Fri Jul 7 16:58:46 UTC 2017


Looks good - did you run existing tests just in case? They are in the 
jdk jtreg test group called jdk_valhalla_mvt

Maurizio


On 07/07/17 17:37, Vladimir Ivanov wrote:
> http://cr.openjdk.java.net/~vlivanov/valhalla/8184017/webrev.00
>
> Q-typed argument binding support. The value is stored in boxed form 
> inside BMH and unboxed on access.
>
> Before: a1:Q
> invoke061_LQL_Q=Lambda(a0:L/SpeciesData<LLLL>,a1:Q,a2:L)=>{
>
> After: t4:Q
>   invoke065_LL_Q=Lambda(a0:L/SpeciesData<LLLLLL>,a1:L)=>{
>     t2:L=BoundMethodHandle$Species_L6.argL5(a0:L);
>     t3:L=BoundMethodHandle$Species_L6.argL4(a0:L);
>     t4:Q=MethodHandle.invokeBasic(t2:L,t3:L);
>
> I haven't added any test case yet, but there will be one in an updated 
> test on vectors.
>
> Best regards,
> Vladimir Ivanov




More information about the valhalla-dev mailing list