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

[Vladimir Ivanov]

On Thu, 1 May 2025 17:15:18 GMT, Igor Veresov <iveresov at openjdk.org> wrote:

>> 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? :)

Nevermind, it turns out to be just a refactoring:`holder == nullptr || !holder->is_loader_present_and_alive()` is equivalent to the original check `!has_cld(holder) || !holder->is_loader_alive()`.

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

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