RFR: 8355915: [leyden] Crash in MDO clearing the unloaded array type [v3]

[Igor Veresov]

On Thu, 1 May 2025 17:07:10 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:

>> Aleksey Shipilev has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision:
>> 
>>  - Co-opt Igor's fix
>>  - Merge branch 'premain' into JDK-8355915-mdo-array-crash
>>  - More robust fix
>>  - Fix
>>  - Fix
>
> src/hotspot/share/ci/ciMethodData.cpp line 82:
> 
>> 80:   bool is_live(Method* m) {
>> 81:     Klass* holder = m->method_holder();
>> 82:     if (holder == nullptr || !holder->is_loader_present_and_alive() || !is_klass_loaded(holder)) {
> 
> Why do we need these checks on CI side?

Wasn't it you who put it there? :)

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

PR Review Comment: https://git.openjdk.org/leyden/pull/64#discussion_r2070542322