[lworld] RFR: 8377451: [lworld] Add ValuePayload abstraction [v8]
Stefan Karlsson
stefank at openjdk.org
Mon Feb 16 10:24:01 UTC 2026
On Mon, 16 Feb 2026 09:42:06 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
>> 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.
We still need to reformat a large portion of the table. Please show how you want this change to be made with minimal formatting.
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/2068#discussion_r2811672813
More information about the valhalla-dev
mailing list