RFR: 8305895: Implement JEP 450: Compact Object Headers (Experimental) [v5]

Roman Kennke rkennke at openjdk.org
Mon Sep 9 13:02:08 UTC 2024


On Fri, 30 Aug 2024 07:42:39 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

>> Yes. This silent setting of UseCompactObjectHeaders ended up hiding why we got CDS failures. I would also suggest that we change this to FLAG_SET_ERGO.
>
> Seems we run all into the same thoughts :)
> 
> I added
> 
> Suggestion:
> 
>     FLAG_SET_DEFAULT(UseCompactObjectHeaders, false);
>     warning("Compact object headers require a java heap size smaller than %zu (given: %zu). "
>                  "Disabling compact object headers.", max_narrow_heap_size * HeapWordSize, max_heap_size);

That %zu is SIZE_FORMAT, right? This should probably use proper_unit_for_byte_size()/byte_size_in_proper_unit().

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20677#discussion_r1750215510


More information about the build-dev mailing list