On Mon, 16 Feb 2026 09:17:20 GMT, Stefan Karlsson <[email protected]> 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

Reply via email to