[lworld] RFR: 8377451: [lworld] Add ValuePayload abstraction [v8]

Stefan Karlsson stefank at openjdk.org
Fri Feb 13 09:41:04 UTC 2026


On Mon, 9 Feb 2026 11:42:07 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

>> Axel Boldt-Christmas has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove mutable
>
> src/hotspot/share/oops/inlineKlassPayload.hpp line 25:
> 
>> 23:  */
>> 24: 
>> 25: #ifndef SHARE_VM_OOPS_INLINEKLASSPAYLOAD_HPP
> 
> Follow-up: Figure out if this should be valuePayload.hpp. Might lead to the discussion about renaming `InlineKlass` to `ValueKlass`, so it might be good to take after integration of this PR, outside of this PR.

After reading through this again and because the main class is called ValuePayload I would prefer if this file where renamed to valuePayload.hpp before this gets pushed.

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/2068#discussion_r2803239482


More information about the valhalla-dev mailing list