On Fri, 24 Apr 2026 13:27:36 GMT, Frederic Parain <[email protected]> wrote:
>> Fixes in the logic to determine if a value class is naturally atomic or not. >> The previous implementation incorrectly considered values classes with a >> single non-empty nullable field as being naturally atomic. >> Also includes improvements to the InstanceKlass::is_naturally_atomic() to >> prevent bad usages. >> >> Tested locally. >> More testing in progress on Mach5 (tiers 1-4). >> >> >> >> --------- >> - [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: > > Comment hard coded arguments to is_naturally_aotimic() LGTM. Thanks for the updates. ------------- Marked as reviewed by phubner (Committer). PR Review: https://git.openjdk.org/valhalla/pull/2361#pullrequestreview-4170810512
