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

Thomas Stuefe stuefe at openjdk.org
Fri Sep 13 12:51:17 UTC 2024


On Tue, 10 Sep 2024 12:35:42 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

>> src/hotspot/share/oops/compressedKlass.cpp line 116:
>> 
>>> 114:   _range = end - _base;
>>> 115: 
>>> 116:   DEBUG_ONLY(assert_is_valid_encoding(addr, len, _base, _shift);)
>> 
>> Can you refactor so the aarch64 path runs this same code without duplication?
>
> In tinycp mode, aarch64 runs this code though? The aarch64 variant of pd_initialize just returns then. In non-COH mode (preexisting, not touched by this patch) Aarch64 needs its own handling.

I refactored: Now we should have no duplication (once my patch hits Romans PR branch)

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

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


More information about the serviceability-dev mailing list