I forgot one important aspect of this patch (how could I!), which is for 
bootstrap method invocation to call invokeWithArguments:

http://cr.openjdk.java.net/~psandoz/jdk10/JDK-8185992-invoke-with-arguments-jumbo/webrev/src/java.base/share/classes/java/lang/invoke/CallSite.java.sdiff.html
 
<http://cr.openjdk.java.net/~psandoz/jdk10/JDK-8185992-invoke-with-arguments-jumbo/webrev/src/java.base/share/classes/java/lang/invoke/CallSite.java.sdiff.html>

I would like to follow up later with some more tests using a separate issue 
(already have some tests in the pipeline). The test for jumbo BSM args may 
depend on whether condy follows or not.

Paul.

> On 18 Aug 2017, at 16:11, Paul Sandoz <paul.san...@oracle.com> wrote:
> 
> Hi,
> 
> Please review this API enhancement for MethodHandle.invokeWithArguments to 
> support jumbo-arity and for
> BSM invocation to be specified in terms of MethodHandle.invokeWithArguments:
> 
>  
> http://cr.openjdk.java.net/~psandoz/jdk10/JDK-8185992-invoke-with-arguments-jumbo/webrev/
> 
> This patch is brought to you by John Rose and initially reviewed by myself.
> 
> It’s low hanging fruit that is the first deliverable from the JEP 309: 
> Dynamic Class-File Constants [1] effort.
> 
> A CSR is underway.
> 
> Paul.
> 
> [1] http://openjdk.java.net/jeps/309
> 
> 

Reply via email to