On Fri, 2 Sep 2022 05:31:35 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.
src/hotspot/share/cds/archiveHeapLoader.cpp line 58: > 56: #include "oops/oop.inline.hpp" > 57: #include "oops/typeArrayOop.inline.hpp" > 58: #include "prims/jvmtiExport.hpp" It looks like you should be able to remove a lot of these header files. ------------- PR: https://git.openjdk.org/jdk/pull/10138
