On Wed, 18 Sep 2024 05:33:09 GMT, David Holmes <dhol...@openjdk.org> wrote:
>> Ioi Lam has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Fixed ZERO build
>
> src/hotspot/share/classfile/systemDictionary.cpp line 139:
>
>> 137: if (_java_platform_loader.is_empty()) {
>> 138: oop platform_loader = get_platform_class_loader_impl(CHECK);
>> 139: _java_platform_loader = OopHandle(Universe::vm_global(),
>> platform_loader);
>
> Why has the order been switched here?
It's just clean up. Platform loader should go before app loader.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20843#discussion_r1766082395
