RFR: 8293336: AOT-linking of invokedynamic for lambda expression and string concat [v4]

Ioi Lam iklam at openjdk.org
Tue Oct 22 06:26:25 UTC 2024


On Mon, 14 Oct 2024 15:54:15 GMT, Dan Heidinga <heidinga at openjdk.org> wrote:

>> Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fixed JDK-8341988: jstack launched with AOT cache created with -XX:+AOTClassLinking crashes
>
> src/hotspot/share/cds/archiveUtils.cpp line 377:
> 
>> 375:   if (SystemDictionaryShared::is_excluded_class(src_ik)) {
>> 376:     return false;
>> 377:   }
> 
> Do we want an assert here that `ArchiveBuilder::current()->get_buffered_addr(src_ik)->has_aot_initialized_mirror() == false`?

These classes are not buffered so `ArchiveBuilder::current()->get_buffered_addr(src_ik)` won't work. I added this assert instead.


  if (SystemDictionaryShared::is_excluded_class(src_ik)) {
    assert(!ArchiveBuilder::current()->has_been_buffered(src_ik), "sanity");
    return false;
  }

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21143#discussion_r1810019800


More information about the core-libs-dev mailing list