On Fri, 2 Sep 2022 18:14:31 GMT, Ioi Lam <[email protected]> wrote: >> I moved all code related to loading the archive heap regions into a new >> file, archiveHeapLoader.cpp. >> >> A diff of the new archiveHeapLoader.cpp and the old heapShared.cpp shows >> that the moved code is identical, except for the change of `HeapShared::` to >> `ArchiveHeapLoader::`. >> >> I also removed unnecessary entries in JVM_EXCLUDE_FILES that are already >> covered by the `cds/` pattern. > > Ioi Lam has updated the pull request incrementally with one additional commit > since the last revision: > > Cleaned up header files
Great! looks good ------------- Marked as reviewed by coleenp (Reviewer). PR: https://git.openjdk.org/jdk/pull/10138
