RFR: 8293293: Move archive heap loading code out of heapShared.cpp
    Coleen Phillimore 
    coleenp at openjdk.org
       
    Fri Sep  2 17:20:53 UTC 2022
    
    
  
On Fri, 2 Sep 2022 05:31:35 GMT, Ioi Lam <iklam at openjdk.org> 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
    
    
More information about the build-dev
mailing list