On Thu, 19 Sep 2024 05:37:17 GMT, David Holmes <dhol...@openjdk.org> wrote:
>> Ioi Lam has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> @dholmes-ora comments
>
> src/hotspot/share/cds/aotClassLinker.cpp line 122:
>
>> 120: assert(CDSConfig::is_dumping_aot_linked_classes(), "sanity");
>> 121:
>> 122: if (!SystemDictionaryShared::is_builtin(ik)) {
>
> What does this actually mean by "built-in"?
Boot/Platform/App loaders. The meaning of "built-in" is documented at the top
of systemDictionaryShared.hpp
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20843#discussion_r1771839512
