RFR 8183130: [MVT] Implement intrinsics for Q-typed LambdaForms

Maurizio Cimadamore maurizio.cimadamore at oracle.com
Mon Jul 3 21:11:48 UTC 2017


I've missed that, sorry - I will remove the assertion

Maurizio


On 03/07/17 22:00, Vladimir Ivanov wrote:
> From correctness POV, it's perfectly fine to miss an intrinsic 
> opportunity. So, I'd prefer to avoid throwing an error until all 
> intrinsics are implemented:
>
> +                default: {
> +                    throw newInternalError("Unknown intrinsic: "+intr);
> +                }
>
> Otherwise, looks good.
>
> Best regards,
> Vladimir Ivanov
>
> On 7/3/17 5:27 PM, Maurizio Cimadamore wrote:
>> Hi,
>> This patch adds support for intrinsic in LambdaFormBuilder. More 
>> intrinsics to come in separate followup patches.
>>
>> http://cr.openjdk.java.net/~mcimadamore/8183130/
>>
>> Thanks
>> Maurizio
>>
>>



More information about the valhalla-dev mailing list