Re: RFR: 8327138: Clean up status management in cdsConfig.hpp and CDS.java [v2]

2024-03-06 Thread Ioi Lam
On Wed, 6 Mar 2024 18:42:01 GMT, Matias Saavedra Silva wrote: >> Ioi Lam has updated the pull request incrementally with one additional >> commit since the last revision: >> >> fixed alignments > > src/hotspot/share/cds/cdsConfig.hpp line 72: > >> 70: static void

Re: RFR: 8327138: Clean up status management in cdsConfig.hpp and CDS.java [v2]

2024-03-06 Thread Ioi Lam
> A few clean ups: > > 1. Rename functions like "`s_loading_full_module_graph()` to > `is_using_full_module_graph()`. The meaning of "loading" is not clear: it > might be interpreted as to cover only the period where the artifact is being > loaded, but not the period after the artifact is