Re: [9] RFR (XS): 8059455: LambdaForm.prepare() does unnecessary work for cached LambdaForms

2015-04-24 Thread Vladimir Ivanov

Thanks, Paul.

Best regards,
Vladimir Ivanov

On 4/23/15 6:30 PM, Paul Sandoz wrote:


On Apr 23, 2015, at 5:12 PM, Vladimir Ivanov vladimir.x.iva...@oracle.com 
wrote:


https://bugs.openjdk.java.net/browse/JDK-8059455
http://cr.openjdk.java.net/~vlivanov/8059455/webrev.00/

LambdaForm.compileToBytecode() does unnecessary work (constructs invokerType  
check an assertion) before ensuring that LambdaForm hasn't been already compiled. 
It happens very often for cached LambdaForms.

The fix is to do the check first.



+1

Paul.


Testing: failed VM tests (timeouts)

Best regards,
Vladimir Ivanov



___
mlvm-dev mailing list
mlvm-dev@openjdk.java.net
http://mail.openjdk.java.net/mailman/listinfo/mlvm-dev


Re: [9] RFR (XS): 8059455: LambdaForm.prepare() does unnecessary work for cached LambdaForms

2015-04-23 Thread Paul Sandoz

On Apr 23, 2015, at 5:12 PM, Vladimir Ivanov vladimir.x.iva...@oracle.com 
wrote:

 https://bugs.openjdk.java.net/browse/JDK-8059455
 http://cr.openjdk.java.net/~vlivanov/8059455/webrev.00/
 
 LambdaForm.compileToBytecode() does unnecessary work (constructs invokerType 
  check an assertion) before ensuring that LambdaForm hasn't been already 
 compiled. It happens very often for cached LambdaForms.
 
 The fix is to do the check first.
 

+1

Paul.

 Testing: failed VM tests (timeouts)
 
 Best regards,
 Vladimir Ivanov



signature.asc
Description: Message signed with OpenPGP using GPGMail
___
mlvm-dev mailing list
mlvm-dev@openjdk.java.net
http://mail.openjdk.java.net/mailman/listinfo/mlvm-dev