RFR: 8350209: Preserve adapters in AOT cache [v7]
Ashutosh Mehra
asmehra at openjdk.org
Thu Apr 24 20:46:51 UTC 2025
On Thu, 24 Apr 2025 01:39:33 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:
>> Vladimir Kozlov has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Fix message
>
> src/hotspot/share/runtime/sharedRuntime.cpp line 2966:
>
>> 2964: adapter_blob = AdapterHandlerLibrary::link_aot_adapter_handler(this);
>> 2965: if (adapter_blob == nullptr) {
>> 2966: log_warning(cds)("Failed to link AdapterHandlerEntry (fp=%s) to its code in the AOT code cache", _fingerprint->as_basic_args_string());
>
> Doesn't it add noise in the output for not yet seen adapter shapes? It's a warning.
`AdapterHandlerEntry::link` method gets called only for the archived adapters. Not yet seen adapters do not come across this code path. Reason for making it a warning is the assumption that, if the AOT Code Cache is usable, then we should always be able to link the `AdapterHandlerEntry` to its code in the AOT code cache. If we fail to do so, then something is not right in this machinery and likely pointing to a bug.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24740#discussion_r2059204247
More information about the hotspot-compiler-dev
mailing list