RFR(XS): 8183233: [MVT] "unexpected value type klass" assert in ValueTypeNode::is_loaded()

Tobias Hartmann tobias.hartmann at oracle.com
Wed Jul 5 08:46:40 UTC 2017


Thanks for the review!

Best regards,
Tobias

On 05.07.2017 10:04, David Simms wrote:
> 
> Looks good
> 
> /Simms
> 
> 
> On 4/07/2017 12:07 p.m., Tobias Hartmann wrote:
>> Hi,
>>
>> please review the following patch:
>> https://bugs.openjdk.java.net/browse/JDK-8183233
>> http://cr.openjdk.java.net/~thartmann/valhalla/vt_prototype/webrev.27/
>>
>> CompileTheWorld hits an assert during compilation when processing the __Value klass. The problem is that C->env()->___Value_klass() is NULL because __Value is not loaded in SystemDictionary::initialize_wk_klass() if !EnableMVT && !EnableValhalla. We should enable MVT if CompileTheWorld is true.
>>
>> Thanks,
>> Tobias
> 


More information about the valhalla-dev mailing list