RFR: 8350916: Remove misleading warning "Cannot dump shared archive while using shared archive"
Calvin Cheung
ccheung at openjdk.org
Fri Feb 28 07:32:57 UTC 2025
On Fri, 28 Feb 2025 02:10:16 GMT, Ioi Lam <iklam at openjdk.org> wrote:
> Please review this trivial change. I replaced an unreachable `warning()` call with an assert and an explanation in comments.
>
> I verified with running all CDS tests and the assert is never triggered.
One typo. Looks good otherwise.
src/hotspot/share/cds/cdsConfig.cpp line 72:
> 70: if (is_dumping_static_archive() && !is_dumping_final_static_archive()) {
> 71: // Note: -Xshare and -XX:AOTMode flags are mutually exclusive.
> 72: // - Class workflow: -Xshare:on and -Xshare:dump cannot take effect at the same time.
s/Class/Classic
-------------
Marked as reviewed by ccheung (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/23836#pullrequestreview-2649964192
PR Review Comment: https://git.openjdk.org/jdk/pull/23836#discussion_r1974935296
More information about the hotspot-runtime-dev
mailing list