On Fri, 17 Apr 2026 13:49:24 GMT, Frederic Parain <[email protected]> wrote:

>> The addition of the NULLABLE_NON_ATOMIC_FLAT layout forced the addition of a 
>> null-marker in all value classes instances, causing a instance size increase 
>> for value classes which didn't supported any nullable layout before. This 
>> size increase caused some performance regressions which boxing classes 
>> Double and Long. This patch temporarily disables the support for the 
>> NULLABLE_NON_ATOMIC_FLAT layout in classes Long and Double. A longer term 
>> solution will be developed and is tracked by JDK-8382361.
>> 
>> Tested on Mach5: tiers 1-4
>> Thanks to @dafedafe for measuring the performance with the fix.
>> 
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Frederic Parain has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Fix typo

Thanks for the fix @fparain! LGTM.

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

Marked as reviewed by dfenacci (Committer).

PR Review: 
https://git.openjdk.org/valhalla/pull/2331#pullrequestreview-4129411959

Reply via email to