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 > >