Integrated: 8292891: ifdef-out some CDS-only functions

Ioi Lam iklam at openjdk.org
Mon Aug 29 23:24:08 UTC 2022


On Wed, 24 Aug 2022 22:27:48 GMT, Ioi Lam <iklam at openjdk.org> wrote:

> 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).

This pull request has now been integrated.

Changeset: 40b0ed56
Author:    Ioi Lam <iklam at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/40b0ed565720fa75fc23e1a8f26e7cfe5feebf8f
Stats:     64 lines in 16 files changed: 41 ins; 19 del; 4 mod

8292891: ifdef-out some CDS-only functions

Reviewed-by: coleenp, ccheung, dholmes

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

PR: https://git.openjdk.org/jdk/pull/10010


More information about the hotspot-dev mailing list