RFR: 8377339: Implement AOTCacheAccess::get_archived_object using heap roots [v2]
Ioi Lam
iklam at openjdk.org
Mon Feb 9 21:59:32 UTC 2026
On Fri, 6 Feb 2026 20:54:18 GMT, Dan Heidinga <heidinga at openjdk.org> wrote:
>> Ioi Lam has updated the pull request incrementally with four additional commits since the last revision:
>>
>> - @DanHeidinga review comment
>> - @DanHeidinga review comment
>> - @DanHeidinga review comment
>> - fixed typo
>
> src/hotspot/share/cds/heapShared.cpp line 256:
>
>> 254: // copying the objects into the cache. This will allow the hashcode to be
>> 255: // dynamically and randomly computed in each production, which generally
>> 256: // desirable for security reasons.
>
> Suggestion:
>
> // desirable to make the hashcodes more random between runs.
Fixed.
> src/hotspot/share/cds/heapShared.cpp line 2468:
>
>> 2466:
>> 2467: ResourceMark rm;
>> 2468: int len;
>
> Suggestion:
>
> int len = 0;
>
> Nitpick: default values make the code easier to reason about.
Fixed
-------------
PR Review Comment: https://git.openjdk.org/leyden/pull/108#discussion_r2783725331
PR Review Comment: https://git.openjdk.org/leyden/pull/108#discussion_r2783728469
More information about the leyden-dev
mailing list