On Fri, 22 May 2026 19:09:15 GMT, Frederic Parain <[email protected]> wrote:
>> Casper Norrbin has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Validate archived inline field dependencies without LoadableDescriptors
>
> src/hotspot/share/classfile/systemDictionary.cpp line 1112:
>
>> 1110: // must be checked against the resolved runtime class before the
>> shared class can be used.
>> 1111: bool
>> SystemDictionary::preload_from_required_inline_field(InstanceKlass* ik,
>> Handle class_loader, Symbol* sig, int field_index, TRAPS) {
>> 1112: TempNewSymbol name = Signature::strip_envelope(sig);
>
> This symbol is only used in logging, could it be possible to create it only
> when a message is logged?
I have now wrapped the creation(s) inside `if (is_log_enabled(...))`.
-------------
PR Review Comment:
https://git.openjdk.org/valhalla/pull/2409#discussion_r3298037228
