RFR: 8332785: Replace naked uses of UseSharedSpaces with CDSConfig::is_using_archive
David Holmes
dholmes at openjdk.org
Fri May 31 05:05:06 UTC 2024
On Wed, 29 May 2024 18:12:25 GMT, Sonia Zaldana Calles <szaldana at openjdk.org> wrote:
> Hi folks,
>
> This PR addresses [8332785](https://bugs.openjdk.org/browse/JDK-8332785) replacing all naked uses for ```UseSharedSpaces``` with ```CDSConfig::is_using_archive```.
>
> Testing:
> - [x] Tier 1 with GHA.
>
> Thanks,
> Sonia
One minor nit but otherwise looks good.
Thanks
src/hotspot/share/cds/cdsConfig.cpp line 308:
> 306:
> 307: bool CDSConfig::has_unsupported_runtime_module_options() {
> 308: assert(CDSConfig::is_using_archive(), "this function is only used with -Xshare:{on,auto}");
Nit: you shouldn't need to specify `CDSConfig::`
-------------
Marked as reviewed by dholmes (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/19463#pullrequestreview-2089898352
PR Review Comment: https://git.openjdk.org/jdk/pull/19463#discussion_r1621718062
More information about the serviceability-dev
mailing list