[lworld] Integrated: 8356963: [lworld] MemNode::optimize_simple_memory_chain fails with "sanity check for basic cases"

Tue May 20 08:17:06 UTC 2025

On Thu, 15 May 2025 16:40:57 GMT, Quan Anh Mai <qamai at openjdk.org> wrote:

> Many thanks to @TobiHartmann for the speedy investigation. This patch fixes the issue by adding a `ciField` for the null marker. I also do a small change to make the name of the `ciField` of a flattened field clearer. I think adding a scenario to test flattening of nullable and atomic fields is necessary. There are some failures that I filed https://bugs.openjdk.org/browse/JDK-8357061 for the investigation.
> 
> Please kindly review, thanks a lot.

This pull request has now been integrated.

Changeset: 8f0e6242
Author:    Quan Anh Mai <qamai at openjdk.org>
Committer: Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.org/valhalla/commit/8f0e62420a27bd0d577da59474fb6936299f775c
Stats:     246 lines in 20 files changed: 126 ins; 68 del; 52 mod

8356963: [lworld] MemNode::optimize_simple_memory_chain fails with "sanity check for basic cases"

Reviewed-by: thartmann

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

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