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