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