[lworld] RFR: 8373912: [lworld] Small cleanups to InlineKlass [v2]

Coleen Phillimore coleenp at openjdk.org
Thu Dec 18 01:29:48 UTC 2025


On Wed, 17 Dec 2025 17:21:21 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

>> A few cleanups found while scanning over the InlineKlass files:
>> 
>> * Fix include list and forward declarations
>> * Move non-trivial implementation out of .hpp
>> * Remove copy-n-past functions from InstanceKlass
>> * Remove redundant CDS function overrides
>> * Deduplicate null marker address calculation
>> * Deduplicate allocate_instance
>> * Fix raw casts from oop 
>> * Tweak comments
>
> Stefan Karlsson has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Remove lingering references to InlineKlassFixedBlock

This looks good.  Thank you for the cleanups.

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

Marked as reviewed by coleenp (Committer).

PR Review: https://git.openjdk.org/valhalla/pull/1818#pullrequestreview-3590298541


More information about the valhalla-dev mailing list