RFR 8183131: [MVT] 2-slot type (long & double) support is broken in LambdaFormBuilder

Vladimir Ivanov vladimir.x.ivanov at oracle.com
Wed Jul 5 12:21:34 UTC 2017


> Let me know what you think
> 
> http://cr.openjdk.java.net/~mcimadamore/8183131_v2/

Nice! I like how it shapes out.

Best regards,
Vladimir Ivanov

> 
> Maurizio
> 
> 
> On 03/07/17 22:13, Maurizio Cimadamore wrote:
>> I see what you mean - I will give this a try
>>
>> Maurizio
>>
>>
>> On 03/07/17 21:53, Vladimir Ivanov wrote:
>>> Looks good.
>>>
>>> Probably, it's better to keep builder.store() wrapped in a helper 
>>> method (as it is done in InvokerBytecodeGenerator) to keep 
>>> localsMap[] conversion implicit.
>>>
>>> Best regards,
>>> Vladimir Ivanov
>>>
>>> On 7/3/17 5:19 PM, Maurizio Cimadamore wrote:
>>>> Hi,
>>>> This patch fixes an issue with lambda form builder not taking into 
>>>> account single vs. double slots indices.
>>>>
>>>> http://cr.openjdk.java.net/~mcimadamore/8183131/
>>>>
>>>> Thanks
>>>> Maurizio
>>>>
>>>>
>>
> 


More information about the valhalla-dev mailing list