RFR: 8362558: [Leyden] fatal error: New workflow: should not compile code for unarchived class: UnsupportedBSMs$$Lambda/0... [v2]

Ioi Lam iklam at openjdk.org
Mon Jul 21 20:37:02 UTC 2025


On Mon, 21 Jul 2025 18:45:24 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   @vnkozlov comments
>
> src/hotspot/share/ci/ciEnv.cpp line 837:
> 
>> 835:     Method* adapter = cpool->resolved_indy_entry_at(index)->method();
>> 836:     if (is_precompiled()) {
>> 837:         ResolvedIndyEntry* indy_info = cpool()->resolved_indy_entry_at(index);
> 
> Can move `indy_info` up and use it in `adapter = ` expression too?
> 
> Do you need to use `()` for `cool()`? Other code does not use it.

I changed `cpool()->` to `cpool->`.

Do you mean this:


if (!AOTConstantPoolResolver::is_resolution_deterministic(cpool(),
         cpool->resolved_indy_entry_at(index)->constant_pool_index())) { 
    adapter = nullptr;
}


I think that will make the code hard to read as you see two "index" in a row.

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

PR Review Comment: https://git.openjdk.org/leyden/pull/87#discussion_r2220258848


More information about the leyden-dev mailing list