RFR(S): 8207844: [lworld] Crash during method resolution because value type holder is not linked

John Rose john.r.rose at oracle.com
Fri Jul 20 22:05:20 UTC 2018


Yes, that's nice.  Thanks.  — John

On Jul 20, 2018, at 2:15 AM, Tobias Hartmann <tobias.hartmann at oracle.com> wrote:
> 
> Hi John,
> 
> thanks for the review.
> 
> On 19.07.2018 22:39, John Rose wrote:
>> Fix is good, but seems a little more cut-n-pasty than I like.
>> Suggest a cleanup which pull all of the conditional logic,
>> plus the deopt, into emit_guard_for_new.
> 
> Yes, that's better. I've pushed a cleanup:
> http://hg.openjdk.java.net/valhalla/valhalla/rev/d6e90a7411bb



More information about the valhalla-dev mailing list