RFR: 8363996: Obsolete UseCompressedClassPointers [v4]

Thomas Stuefe stuefe at openjdk.org
Mon Feb 9 21:36:43 UTC 2026


On Mon, 9 Feb 2026 11:11:24 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

>> src/hotspot/share/oops/oop.inline.hpp line 157:
>> 
>>> 155:   assert(Universe::is_bootstrapping() || (k != nullptr && k->is_klass()), "incorrect Klass");
>>> 156:   assert(!UseCompactObjectHeaders, "don't set Klass* with compact headers");
>>> 157:   _metadata._compressed_klass = CompressedKlassPointers::encode_not_null(k);
>> 
>> We might have to reconsider if we need to maintain the`union _metadata`
>
> Right you are. We only should need narrowKlass now, even on 32-bit.

Fixed: removed union, replaced with narrowKlass

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28366#discussion_r2782264128


More information about the core-libs-dev mailing list