Integrated: 8352696: JFR: assert(false): EA: missing memory path

[Markus Grönlund]

On Mon, 24 Mar 2025 11:35:03 GMT, Markus Grönlund <mgronlun at openjdk.org> wrote:

> Greetings,
> 
> In debug builds, we can sometimes assert because C2's Escape Analysis does not recognize a pattern where one input of memory Phi node is a MergeMem node, and another is a RAW store.
> 
> This pattern is created by the jdk.jfr.internal.JVM.commit() intrinsic, which is inlined because of inlining JFR events, for example jdk.VirtualThreadStart.
> 
> As a result, EA complains about a strange memory graph. 
> 
> Testing: jdk_jfr
> 
> Thanks
> Markus

This pull request has now been integrated.

Changeset: 721ef767
Author:    Markus Grönlund <mgronlun at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/721ef76738a2145bdff9b8534d3512282c61db8b
Stats:     101 lines in 2 files changed: 94 ins; 2 del; 5 mod

8352696: JFR: assert(false): EA: missing memory path

Reviewed-by: thartmann, shade, kvn

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

PR: https://git.openjdk.org/jdk/pull/24192