Re: [9] RFR (M): 8050877: Improve code for pairwise argument conversions and value boxing/unboxing

2014-07-18 Thread Vladimir Ivanov
Thanks for review, Paul. MethodHandleImpl Merge the the 'if' into an 'else if': 365 } else { 366 if (dst.isPrimitive()) { Done. ValueConversions I can see why an EnumMap is used for convenience mapping the Wrapper to MH. IIUC it means the MH ref values are not @Sta

Re: [9] RFR (M): 8050884: Intrinsify ValueConversions.identity() functions

2014-07-18 Thread Vladimir Ivanov
Thanks for review, Paul. Looks good. Same question as in previous email on @Stable for MethodHandles.IDENTITY_MHS. FWIW for MethodHandles.IDENTITY_MHS the Wrapper.ordinal() is used as an index rather than using an EnumMap. Yes, caching logic is inconsistent across java.lang.invoke. I'll try t