RFR(S) JDK-8206140 [lworld] Move return value null checks into the callee

Ioi Lam ioi.lam at oracle.com
Thu Jul 12 19:19:55 UTC 2018


I've simplified the runtime part of this patch after Harold pushed the 
"Add value types consistency checks" changes.

http://cr.openjdk.java.net/~iklam/valhalla/8206140_lworld_null_check_in_callee.v04/

Now Method::is_returning_vt() is initialized during class preparation 
time, and the run-time checks in the interpreter is kept to a minimum.

There's no change to the compiler part of the patch since the last webrev.

The test cases have been simplified since I no longer need to test for 
value types consistency.

Thanks

- Ioi



On 7/10/18 8:59 AM, Ioi Lam wrote:
> Oops, will do :-)
>
> Thanks
>
> - Ioi
>
>
> On 7/10/18 8:47 AM, Tobias Hartmann wrote:
>> Hi Ioi,
>>
>> On 10.07.2018 17:33, Ioi Lam wrote:
>>> http://cr.openjdk.java.net/~iklam/valhalla/8206140_lworld_null_check_in_callee.v03/ 
>>>
>> Looks good to me but I think you need to add the copyright headers to 
>> the .jasm files (no new webrev
>> required).
>>
>> Thanks,
>> Tobias
>




More information about the valhalla-dev mailing list