On Jul 15, 2014, at 3:48 PM, Vladimir Ivanov
wrote:
> Update: http://cr.openjdk.java.net/~vlivanov/8050053/webrev.01
> Diff: http://cr.openjdk.java.net/~vlivanov/8050053/webrev.diff.00-01/
>
> Got rid of varargs & spread invokers.
>
+1
Paul.
signature.asc
Description: Message signed with
On Jul 15, 2014, at 3:42 PM, Vladimir Ivanov
wrote:
> Another update:
> http://cr.openjdk.java.net/~vlivanov/8050052//webrev.01/
>
> Tentative diff:
> http://cr.openjdk.java.net/~vlivanov/8050052/webrev.diff.01-02/
>
+1
Paul.
signature.asc
Description: Message signed with OpenPGP using GP
Thank you, Paul.
Best regards,
Vladimir Ivanov
On 7/16/14 11:28 AM, Paul Sandoz wrote:
On Jul 15, 2014, at 3:48 PM, Vladimir Ivanov
wrote:
Update: http://cr.openjdk.java.net/~vlivanov/8050053/webrev.01
Diff: http://cr.openjdk.java.net/~vlivanov/8050053/webrev.diff.00-01/
Got rid of vararg
Thank you, Paul.
Best regards,
Vladimir Ivanov
On 7/16/14 11:35 AM, Paul Sandoz wrote:
On Jul 15, 2014, at 3:42 PM, Vladimir Ivanov
wrote:
Another update:
http://cr.openjdk.java.net/~vlivanov/8050052//webrev.01/
Tentative diff:
http://cr.openjdk.java.net/~vlivanov/8050052/webrev.diff.01-0
On Jul 14, 2014, at 4:04 PM, Vladimir Ivanov
wrote:
> http://cr.openjdk.java.net/~vlivanov/8050166/webrev.00/
> https://bugs.openjdk.java.net/browse/JDK-8050166
>
> Get rid of the following methods in j.l.i.MethodHandle:
> * convertArguments(MethodType newType)
> * bindArgument(int pos, Basi
On Jul 14, 2014, at 4:37 PM, Vladimir Ivanov
wrote:
> http://cr.openjdk.java.net/~vlivanov/8050173/webrev.00/
> https://bugs.openjdk.java.net/browse/JDK-8050173
>
> Added j.l.i.MethodHandle.copyWith(MethodType, LambdaForm) and provided
> implementation for all subclasses.
>
> Also, some clea
On Jul 14, 2014, at 4:47 PM, Vladimir Ivanov
wrote:
> http://cr.openjdk.java.net/~vlivanov/8050174/webrev.00/
> https://bugs.openjdk.java.net/browse/JDK-8050174
>
> Support overriding of isInvokeSpecial flag in WrappedMember.
>
> Testing: jdk/java/lang/invoke, jdk/java/util/streams, nashorn,
On Jul 14, 2014, at 5:17 PM, Vladimir Ivanov
wrote:
> http://cr.openjdk.java.net/~vlivanov/8050057/webrev.00/
> https://bugs.openjdk.java.net/browse/JDK-8050057
>
> Cache MethodHandle reinvokers per basic type.
> For BoundMethodHandles, rebinding is no-op unless underlying LF is too
> complex
On Jul 14, 2014, at 7:10 PM, Vladimir Ivanov
wrote:
> http://cr.openjdk.java.net/~vlivanov/8050200/webrev.00/
> https://bugs.openjdk.java.net/browse/JDK-8050200
>
> Replace pattern matching sequences of LambdaForm names during compilation
> with explicit marks on MethodHandles. Intrinsic ID i
Thank you, Paul.
Looks good. Juste one comment.
BoundMethodHandle
The following fields can be made final:
132 private static int FIELD_COUNT_THRESHOLD = 12; // largest convenient
BMH field count
133 private static int FORM_EXPRESSION_THRESHOLD = 24; // largest
convenient BMH expres
Paul, thanks for review.
On 7/16/14 12:34 PM, Paul Sandoz wrote:
On Jul 14, 2014, at 4:04 PM, Vladimir Ivanov
wrote:
http://cr.openjdk.java.net/~vlivanov/8050166/webrev.00/
https://bugs.openjdk.java.net/browse/JDK-8050166
Get rid of the following methods in j.l.i.MethodHandle:
* convertA
Paul, thanks for review.
http://cr.openjdk.java.net/~vlivanov/8050200/webrev.00/
https://bugs.openjdk.java.net/browse/JDK-8050200
Replace pattern matching sequences of LambdaForm names during compilation with
explicit marks on MethodHandles. Intrinsic ID is associated with a method
handle usi
On Jul 16, 2014, at 12:53 PM, Vladimir Ivanov
wrote:
> Paul, thanks for review.
>
> On 7/16/14 12:34 PM, Paul Sandoz wrote:
>>
>> On Jul 14, 2014, at 4:04 PM, Vladimir Ivanov
>> wrote:
>>
>>> http://cr.openjdk.java.net/~vlivanov/8050166/webrev.00/
>>> https://bugs.openjdk.java.net/browse/J
http://cr.openjdk.java.net/~vlivanov/8050877/webrev.00/
https://bugs.openjdk.java.net/browse/JDK-8050877
Improved MethodHandleImpl.makePairwiseConvert & ValueConversions.unbox
and small cleanups in related code.
Also, improved method handle caching in ValueConversions.
MethodHandleImpl.makePa
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
http://cr.openjdk.java.net/~vlivanov/8050887/webrev.00
https://bugs.openjdk.java.net/browse/JDK-8050887
Intrinsify MethodHandles.constant() for default values.
Testing: jdk/java/lang/invoke, jdk/java/util/streams, nashorn, octane w/
"-ea -esa" and COMPILE_THRESHOLD={0,30}.
Reviewed-by: vlivan
Remi,
The problem is that for every iteration you create new call site
> for(int i=0; i<100_000; i++) {
>new LoopyCS().getTarget().invokeExact(1_000);
> }
In LoopyCS constructor you instantiates 3 new MethodHandles:
>target = MethodHandles.filterArguments(target, 0, FOO
On 07/16/2014 07:38 PM, Vladimir Ivanov wrote:
Remi,
The problem is that for every iteration you create new call site
> for(int i=0; i<100_000; i++) {
>new LoopyCS().getTarget().invokeExact(1_000);
> }
In LoopyCS constructor you instantiates 3 new MethodHandles:
>targ
On Jul 16, 2014, at 11:40 AM, Remi Forax wrote:
>
> On 07/16/2014 07:38 PM, Vladimir Ivanov wrote:
>> Remi,
>>
>> The problem is that for every iteration you create new call site
>> > for(int i=0; i<100_000; i++) {
>> >new LoopyCS().getTarget().invokeExact(1_000);
>> > }
>>
19 matches
Mail list logo
