RFR(S) JDK-8206140 [lworld] Move return value null checks into the callee
tobias.hartmann at oracle.com
Fri Jul 13 12:20:45 UTC 2018
thanks for making these changes!
Your latest version of the patch looks good to me but I think we should hold off with pushing until
we know how to fix JDK-8207219 and fully understand if value attribute consistency checking is
strong enough to enable this optimization in all cases.
On 12.07.2018 21:19, Ioi Lam wrote:
> I've simplified the runtime part of this patch after Harold pushed the "Add value types consistency
> checks" changes.
> 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.
> - Ioi
> On 7/10/18 8:59 AM, Ioi Lam wrote:
>> Oops, will do :-)
>> - Ioi
>> On 7/10/18 8:47 AM, Tobias Hartmann wrote:
>>> Hi Ioi,
>>> On 10.07.2018 17:33, Ioi Lam wrote:
>>> Looks good to me but I think you need to add the copyright headers to the .jasm files (no new webrev
More information about the valhalla-dev