RFR: 8344363: FullGCForwarding::initialize_flags is called after ObjLayout::initialize [v2]

[Roman Kennke]

On Wed, 20 Nov 2024 15:42:46 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

> So we are inlining `initialize_heap_flags_and_sizes` into `initialize`, right?

No, not really. `initialize_heap_flags_and_sizes` is still called from `initialize_heap_sizes` which is called from `universe_init`.

> Is `ParallelArguments::initialize_heap_flags_and_sizes()` dangling now?

No.
 
> It sounds like there is a little bit of symmetry between `CompressedKlassPointers::initialize` that is called from `universe_init`, and thus after argument adjustments. I don't know if it makes sense to move `ObjLayout::initialize` to `universe_init`? 

Hmm yeah, that makes sense. I'll do that.

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

PR Comment: https://git.openjdk.org/jdk/pull/22255#issuecomment-2489412764