On Tue, 31 Mar 2026 08:10:53 GMT, Paul Hübner <[email protected]> wrote:
>> src/hotspot/share/cds/heapShared.cpp line 123:
>>
>>> 121: static const char* _archived_cache_name = "archivedCache";
>>> 122: static ArchivableStaticFieldInfo archive_subgraph_entry_fields[] = {
>>> 123: {"java/lang/Integer$IntegerCache",
>>> _archived_cache_name},
>>
>> I don't think this change is necessary, we should avoid unneeded diffs if
>> possible.
>
> It's used to re-use `_archived_cache_name` in the new assertion later on.
> Would you like me to revert this assertion?
Or should I use the `"archivedCache"` literal twice?
-------------
PR Review Comment:
https://git.openjdk.org/valhalla/pull/2249#discussion_r3014252006
