[lworld] RFR: 8355397: [lworld] Parse::array_load fails with "array can't be flat" assert [v2]
    Quan Anh Mai 
    qamai at openjdk.org
       
    Mon Apr 28 14:46:42 UTC 2025
    
    
  
> Hi,
> 
> The issue here is that `InstanceKlass::flat_array` is too strict and it rejects any class that is atomic and has more than 1 field. I remove the unnecessary restrictions and rename the method to `maybe_flat_in_array` to illustrate the functionality in a clearer manner.
> 
> Please kindly review, thanks a lot.
Quan Anh Mai has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
 - Merge branch 'lworld' into flatinarray
 - add tests
 - fix InstanceKlass::flat_array
-------------
Changes:
  - all: https://git.openjdk.org/valhalla/pull/1446/files
  - new: https://git.openjdk.org/valhalla/pull/1446/files/fcfd9334..55e444a6
Webrevs:
 - full: https://webrevs.openjdk.org/?repo=valhalla&pr=1446&range=01
 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=1446&range=00-01
  Stats: 304 lines in 6 files changed: 278 ins; 2 del; 24 mod
  Patch: https://git.openjdk.org/valhalla/pull/1446.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/1446/head:pull/1446
PR: https://git.openjdk.org/valhalla/pull/1446
    
    
More information about the valhalla-dev
mailing list