[lworld] RFR: 8373706: [lworld] Layout names should be self-explanatory [v2]

Frederic Parain fparain at openjdk.org
Tue Dec 16 14:52:57 UTC 2025


On Mon, 15 Dec 2025 22:32:09 GMT, Dan Heidinga <heidinga at openjdk.org> wrote:

>> Frederic Parain has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix ending blank line
>
> looks reasonable to me

Thanks @DanHeidinga @MrSimms for your reviews.

> src/hotspot/share/oops/layoutKind.cpp line 45:
> 
>> 43:       ShouldNotReachHere();
>> 44:   }
>> 45: }
> 
> Do we need a blank line at the end of the file?

Fixed

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

PR Comment: https://git.openjdk.org/valhalla/pull/1801#issuecomment-3660934478
PR Review Comment: https://git.openjdk.org/valhalla/pull/1801#discussion_r2623590406


More information about the valhalla-dev mailing list