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

Aleksey Shipilev shade at openjdk.org
Mon Feb 16 09:44:42 UTC 2026


On Mon, 16 Feb 2026 09:17:20 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

>> Yeah, please keep the table formatting intact. For one thing, it obscures what you are semantically changing in the table? I understand it would be uglier, but that's OK.
>
> To be clear, do you want the table to be like this change (which includes alignment fixes and comment fixes):
> https://github.com/openjdk/valhalla/pull/2068/changes/59c1fc2e7dedd079a61dced4401267dace7c642d
> 
> Or do you have some other formatting in mind?

Well, that's my point: the semantics-changing PR _should not_ include large formatting changes like this, as it obscures the thing being done. Do the minimal (ugly) thing of adding/modifying things you need for this PR to work.

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

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


More information about the valhalla-dev mailing list