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

Vladimir Ivanov vladimir.x.ivanov at oracle.com
Mon Jul 3 21:00:28 UTC 2017


 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