On Wed, 16 Apr 2025 16:44:05 GMT, Calvin Cheung <cche...@openjdk.org> wrote:
>> Ioi Lam 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: >> >> - Merge branch 'master' into >> 8352001-exclude-injected-classes-from-builtin-loaders >> - @calvinccheung review >> - @matias9927 comment >> - Merge branch 'master' into >> 8352001-exclude-injected-classes-from-builtin-loaders >> - 8352001: AOT cache should not contain classes injected into built-in >> class loaders > > src/hotspot/share/cds/aotArtifactFinder.cpp line 26: > >> 24: >> 25: #include "cds/aotClassLinker.hpp" >> 26: #include "cds/aotClassLocation.hpp" > > Is this include needed? I added it by mistake. I've removed it. Could you re-approve? ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/24046#discussion_r2047374885
