RFR: 8292891: ifdef-out some CDS-only functions [v5]
Ioi Lam
iklam at openjdk.org
Mon Aug 29 20:52:21 UTC 2022
> Some CDS functions are always compiled, even when CDS is not enabled (e.g., for the minimal VM).
>
> This RFE puts some of the obvious ones inside `#if INCLUDE_CDS` blocks.
>
> Note: my goal is not to make the minimal VM as small as possible. But rather, I don't want to put `#if INCLUDE_CDS` inside each of those functions that access a CDS-only feature. (E.g., `ConstantPoolCache::save_for_archive`, which accesses `_initial_entries`, which is declared only when CDS is enabled).
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 five additional commits since the last revision:
- Merge branch 'master' of https://github.com/openjdk/jdk into 8292891-ifdef-out-some-cds-functions
- fixed typo
- @dholmes-ora and @coleenp comments
- @calvinccheung comments
- 8292891: ifdef-out some CDS-only functions
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/10010/files
- new: https://git.openjdk.org/jdk/pull/10010/files/5f8bcd39..7f80b1a5
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=10010&range=04
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=10010&range=03-04
Stats: 15931 lines in 452 files changed: 6249 ins; 7912 del; 1770 mod
Patch: https://git.openjdk.org/jdk/pull/10010.diff
Fetch: git fetch https://git.openjdk.org/jdk pull/10010/head:pull/10010
PR: https://git.openjdk.org/jdk/pull/10010
More information about the hotspot-dev
mailing list