On Wed, 17 Dec 2025 15:18:04 GMT, Quan Anh Mai <[email protected]> wrote:
>> Hi, >> >> This PR fixes various issues with `TypeAryPtr` and `TypeAryKlassPtr`. It >> starts with me trying to tighten the properties (`flat`, `null_free`, etc) >> of these classes, then fixing all the revealed issues until there are no >> crashes or wrong results left. >> >> Please take a look and leave your reviews, thanks a lot. > > Quan Anh Mai has updated the pull request with a new target base due to a > merge or a rebase. The pull request now contains 14 commits: > > - ArrayProperties needs to agree with LayoutKind > - Merge branch 'lworld' into AryKlassPtrfactory > - Merge branch 'lworld' into AryKlassPtrfactory > - reviews > - reviews > - Should not clone MergeMem > - fix NarrowMemProj > - Merge branch 'lworld' into AryKlassPtrfactory > - Fix Parse::array_store and PhaseMacroNode::value_from_mem > - whitespace > - ... and 4 more: > https://git.openjdk.org/valhalla/compare/1077e4f9...bdd91920 I have resolved the merge conflict and the failure. The issue was that the properties of an `ArrayKlass` do not agree with its layout kind. What I did was to normalize the properties after layout selection. @fparain What do you think? Thanks a lot. ------------- PR Comment: https://git.openjdk.org/valhalla/pull/1755#issuecomment-3666442763
