RFR: 8265696: Move CDS sources to src/hotspot/shared/cds [v6]
Ioi Lam
iklam at openjdk.java.net
Fri Apr 23 03:07:02 UTC 2021
> 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 11 additional commits since the last revision:
- Merge branch 'master' into 8265696-move-cds-sources
- fixed merge
- Merge branch 'master' into 8265696-move-cds-sources
- Merge branch 'master' into 8265696-move-cds-sources
- 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
- ... and 1 more: https://git.openjdk.java.net/jdk/compare/ba1998d8...8a9e7bdf
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/3610/files
- new: https://git.openjdk.java.net/jdk/pull/3610/files/5ca1c512..8a9e7bdf
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3610&range=05
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3610&range=04-05
Stats: 2 lines in 1 file changed: 2 ins; 0 del; 0 mod
Patch: https://git.openjdk.java.net/jdk/pull/3610.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/3610/head:pull/3610
PR: https://git.openjdk.java.net/jdk/pull/3610
More information about the build-dev
mailing list