On Wed, 9 Sep 2026 21:08:23 GMT, Matias Saavedra Silva <[email protected]>
wrote:
>> Ioi Lam has updated the pull request incrementally with two additional
>> commits since the last revision:
>>
>> - clean up: ValuePayloadContext::_klass is used only in asserts
>> - Alternative
>
> src/hotspot/share/cds/aotMapLogger.cpp line 1050:
>
>> 1048: st->cr();
>> 1049: ValuePayloadContext vpc{elem_k, elem_offset};
>> 1050: ArchivedFieldPrinter print_field(fake_flat_array, st, 0, &vpc);
>
> Indentation is not preserved from before your change. Previously indentation
> started at 1 so fields were indented one level from the holder.
Good catch!
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/32565#discussion_r3978460360
