RFR: 8265696 move cds sources [v3]

[David Holmes]

On Thu, 22 Apr 2021 06:27:45 GMT, Ioi Lam <iklam at 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.
>
> 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 seven additional commits since the last revision:
> 
>  - Merge branch 'master' into 8265696-move-cds-sources
>  - exclude all files under shared/cds if CDS is disabled; compactHashtable.cpp cannot be excluded since a bit of it is used even when CDS is disabled
>  - fixed include guards -> #ifndef SHARE_CDS_xxxxx
>  - fixed copyright
>  - moved more files
>  - fixed include lines
>  - 8265696: Move CDS sources from shared/memory to shared/cds

Updates look good.

Thanks,
David

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

Marked as reviewed by dholmes (Reviewer).

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