[lworld] [Rev 01] RFR: 8244810: [lworld] Decommission arrayStorageProperties from Klass:array_klass() et el

David Simms dsimms at openjdk.java.net
Thu May 14 06:43:09 UTC 2020


On Wed, 13 May 2020 15:54:18 GMT, Frederic Parain <fparain at openjdk.org> wrote:

>> David Simms has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Copyright syntax
>
> src/hotspot/share/memory/oopFactory.cpp line 155:
> 
>> 154:   ValueArrayKlass* vak = ValueArrayKlass::cast(array->klass());
>> 155:   objArrayHandle oarray = new_objArray_handle(vak->element_klass()->super(),
>> 156:                                               array->length(), CHECK_(objArrayHandle()));
> 
> With the evolving model, it is not guaranteed that the direct super type will not be an inline type, it might be safer
> to create an `Object[]` instead.

Totally agree.

-------------

PR: https://git.openjdk.java.net/valhalla/pull/44


More information about the valhalla-dev mailing list