[9] RFR (M): 8037209: Improvements and cleanups to bytecode assembly for lambda forms

[Paul Sandoz]

On Jul 8, 2014, at 12:09 PM, Vladimir Ivanov <vladimir.x.ivanov at oracle.com> wrote:
> I'd like to revive this review thread.
> 
> Updated version:
> http://cr.openjdk.java.net/~vlivanov/8037209/webrev.04/
> 

Seems ok. I don't claim to be knowledgable enough in the finer points of conversion/casting/verification but i could see anything obviously wrong with the code.


> Paul, I'd like to address your case (if possible) separately. Right now, I don't see any way to avoid null check you see with your tests.
> 

Ok, thanks. I was naively pondering whether it would be worth while replacing the cast with an intrinsic Unsafe.typeProfile(Class c, Object o, boolean isNullInteresting), then that could be called with (c, instance, false), although dunno if that would be any easier to solve.

Paul.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 841 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <http://mail.openjdk.java.net/pipermail/mlvm-dev/attachments/20140708/ca245026/signature.asc>