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

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

2014-07-17 Thread Paul Sandoz
On Jul 16, 2014, at 6:44 PM, Vladimir Ivanov wrote: > http://cr.openjdk.java.net/~vlivanov/8050884/webrev.00/ > https://bugs.openjdk.java.net/browse/JDK-8050884 > > Replace ValueConversions.identity() functions with intrinsics. > > Testing: jdk/java/lang/invoke, jdk/java/util/streams, nashorn,

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

2014-07-16 Thread Vladimir Ivanov
http://cr.openjdk.java.net/~vlivanov/8050884/webrev.00/ https://bugs.openjdk.java.net/browse/JDK-8050884 Replace ValueConversions.identity() functions with intrinsics. Testing: jdk/java/lang/invoke, jdk/java/util/streams, nashorn, octane w/ "-ea -esa" and COMPILE_THRESHOLD={0,30}. Reviewed-by