RFR: 8292891: ifdef-out some CDS-only functions [v3]
Ioi Lam
iklam at openjdk.org
Fri Aug 26 04:45:32 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 incrementally with one additional commit since the last revision:
@dholmes-ora and @coleenp comments
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/10010/files
- new: https://git.openjdk.org/jdk/pull/10010/files/9941eb7e..17a20bde
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=10010&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=10010&range=01-02
Stats: 5 lines in 2 files changed: 2 ins; 0 del; 3 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