review request (L): 7013417: JSR 292 needs to support variadic method handle calls

John Rose john.r.rose at oracle.com
Thu Jan 20 12:36:44 PST 2011


Oops; one more change.  (Didn't run the unit tests properly.)

http://cr.openjdk.java.net/~jrose/7013417/webrev.01

Three files changed:

 - http://cr.openjdk.java.net/~jrose/7013417/webrev.01/src/share/classes/sun/dyn/CallSiteImpl.java.udiff.html
  (one char diff in last hunk: s/TRANSITIONAL/!TRANSITIONAL/)

 - http://cr.openjdk.java.net/~jrose/7013417/webrev.01+/test/java/dyn/InvokeDynamicPrintArgs.java.udiff.html
  (updated example and smoke test:  now the smoke comes out of the right places)

 - http://cr.openjdk.java.net/~jrose/7013417/webrev.01/test/java/dyn/indify/Indify.java.udiff.html
  (updated copy of Indify)

-- John

On Jan 20, 2011, at 12:03 PM, John Rose wrote:

> On Jan 20, 2011, at 11:56 AM, Christian Thalinger wrote:
> 
>> Looks good.  -- Christian
> 
> Thanks.   And thanks for the detailed pre-review based on the mlvm patch.  -- John



More information about the hotspot-compiler-dev mailing list