Review Request: JDK-8186874: [MVT] Method name generated by ValueType is illegal

Maurizio Cimadamore maurizio.cimadamore at oracle.com
Tue Sep 19 23:46:52 UTC 2017


Looks great - thanks!

Maurizio


On 20/09/17 00:26, mandy chung wrote:
> Webrev:
> http://cr.openjdk.java.net/~mchung/valhalla/webrevs/8186874/webrev.00/
>
> This patch fixes the bytecode generation with a legal method name.  I 
> test with the runtime/valuetypes and compiler/valuetypes tests with 
> -noverify flag removed except ValueOops.java and VboxUnbox.java tests. 
> ValueOops.java and VboxUnbox.java tests fail with a different error 
> when verifier is enabled.  I'll look into JDK-8187696 separately.
>
> Tobias - the webrev includes your patch for JDK-8187670 to verify my 
> fix.  Do you prefer to leave it for you to push separately?  Or I can 
> push it for you?
>
> thanks
> Mandy




More information about the valhalla-dev mailing list