RFR: 8350209: Preserve adapters in AOT cache [v4]

Vladimir Kozlov kvn at openjdk.org
Tue Apr 22 17:28:12 UTC 2025


On Tue, 22 Apr 2025 17:01:20 GMT, Ioi Lam <iklam at openjdk.org> wrote:

>> Vladimir Kozlov has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Add sanity test for AOTAdapterCaching flag
>
> src/hotspot/share/cds/cdsConfig.cpp line 869:
> 
>> 867: 
>> 868: bool CDSConfig::is_dumping_aot_code_enabled() {
>> 869:   return _is_dumping_aot_code_enabled;
> 
> Other functions in CDSConfig don't have the `_enabled()` suffix. E.g., `is_dumping_method_handles()`. It doesn't mean we are doing method handle right now, but rather we have the ability to do so.
> 
> Since we rarely ask "am I in the middle of dumping X right now", I think adding `_enabled()` will be redundant.

Okay.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/24740#discussion_r2054541461


More information about the hotspot-compiler-dev mailing list