[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