On Fri, 22 May 2026 17:04:21 GMT, Coleen Phillimore <[email protected]> wrote:

>> src/hotspot/share/classfile/systemDictionaryShared.cpp line 313:
>> 
>>> 311:           Handle loader(current, k->class_loader());
>>> 312:           Symbol* field_klass_name = 
>>> Signature::strip_envelope(fs.signature());
>>> 313:           Klass* field_klass = 
>>> SystemDictionary::find_instance_or_array_klass(current, field_klass_name, 
>>> loader);
>> 
>> Why doing a system dictionary search for flat fields when a pointer is 
>> already directly available in the _inline_layout_info_array?
>
> if the field is a null_free_inline_type is it also available in the 
> _inline_layout_info_array?
> This search is pretty quick so would be good to have them done the same way.

I was hoping the klasses were cached somewhere but I forgot about that array. 
Thanks for pointing it out!

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

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2464#discussion_r3290099504

Reply via email to