> 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:

  Add test with empty values

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

Changes:
  - all: https://git.openjdk.org/valhalla/pull/2361/files
  - new: https://git.openjdk.org/valhalla/pull/2361/files/09fabb75..4c574925

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=valhalla&pr=2361&range=02
 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=2361&range=01-02

  Stats: 34 lines in 1 file changed: 33 ins; 1 del; 0 mod
  Patch: https://git.openjdk.org/valhalla/pull/2361.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/2361/head:pull/2361

PR: https://git.openjdk.org/valhalla/pull/2361

Reply via email to