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

Axel Boldt-Christmas aboldtch at openjdk.org
Mon Feb 16 11:01:32 UTC 2026


On Mon, 16 Feb 2026 10:37:42 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> We still need to reformat a large portion of the table. Please show how you want this change to be made with minimal formatting.
>
>> We still need to reformat a large portion of the table 
> 
> I do not understand this part. What/who says you *need* to reformat the table? You can certainly avoid doing whitespace changes, can you not? I have just spot-checked a few lines in current diff, and most of the lines I tried contain only whitespace changes. Which makes the diff larger than it can/should be.

Is what you want this https://github.com/openjdk/valhalla/commit/b38f3672d1d2c2f3ecd7b8b48db7be29148662fa with some effort to keep the table sane, but the columns are unaligned?

Or this with absolutely no whitespace changes? ~~https://github.com/openjdk/valhalla/commit/2dba3de2331b6fdb2148c2fc48b4d514045b26f9~~ https://github.com/openjdk/valhalla/commit/d39601ddf6b4610a905465f3ea97b01bd6254cc1

Or https://github.com/openjdk/valhalla/commit/59c1fc2e7dedd079a61dced4401267dace7c642d which is the first version but the columns aligned. (When modifying structured / aligned tables, variable declarations / definitions, I usually try to fix their format after the change.)

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

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


More information about the valhalla-dev mailing list