On Wed, 21 Apr 2021 21:55:25 GMT, Ioi Lam <ik...@openjdk.org> wrote:

> The number of CDS source files have grown significantly. To improve 
> modularity, the following files should be moved a new directory, 
> src/hotspot/share/cds.
> 
> - src/hotspot/share/classfile/classListParser.cpp
> - src/hotspot/share/classfile/classListParser.hpp
> - src/hotspot/share/classfile/classListWriter.hpp
> - src/hotspot/share/classfile/compactHashtable.cpp
> - src/hotspot/share/classfile/compactHashtable.hpp
> - src/hotspot/share/classfile/lambdaFormInvokers.cpp
> - src/hotspot/share/classfile/lambdaFormInvokers.hpp
> - src/hotspot/share/memory/archiveBuilder.cpp
> - src/hotspot/share/memory/archiveBuilder.hpp
> - src/hotspot/share/memory/archiveUtils.cpp
> - src/hotspot/share/memory/archiveUtils.hpp
> - src/hotspot/share/memory/archiveUtils.inline.hpp
> - src/hotspot/share/memory/cppVtables.cpp
> - src/hotspot/share/memory/cppVtables.hpp
> - src/hotspot/share/memory/dumpAllocStats.cpp
> - src/hotspot/share/memory/dumpAllocStats.hpp
> - src/hotspot/share/memory/dynamicArchive.cpp
> - src/hotspot/share/memory/dynamicArchive.hpp
> - src/hotspot/share/memory/filemap.cpp
> - src/hotspot/share/memory/filemap.hpp
> - src/hotspot/share/memory/heapShared.cpp
> - src/hotspot/share/memory/heapShared.hpp
> - src/hotspot/share/memory/heapShared.inline.hpp
> - src/hotspot/share/memory/metaspaceShared.cpp
> - src/hotspot/share/memory/metaspaceShared.hpp
> - src/hotspot/share/prims/cdsoffsets.cpp
> - src/hotspot/share/prims/cdsoffsets.hpp
> 
> Testing with mach5: tier1, builds-tier2, builds-tier3, builds-tier4 and 
> builds-tier5. Also locally: aarch64, arm, ppc64, s390, x86, and zero.

make/hotspot/lib/JvmFeatures.gmk line 134:

> 132:       metaspaceShared_$(HOTSPOT_TARGET_CPU_ARCH).cpp \
> 133:       sharedClassUtil.cpp \
> 134:       sharedPathsMiscInfo.cpp \

Removed obsolete files that no longer exist.

src/hotspot/os/windows/os_windows.cpp line 42:

> 40: #include "logging/logStream.hpp"
> 41: #include "memory/allocation.inline.hpp"
> 42: #include "oops/oop.inline.hpp"

Removed some unnecessary inclusions.

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

PR: https://git.openjdk.java.net/jdk/pull/3610

Reply via email to