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

[Tobias Hartmann]

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

test/jdk/jdk/jfr/jvm/TestJvmCommitIntrinsicAndEA.java line 85:

> 83: public final class TestJvmCommitIntrinsicAndEA {
> 84: 
> 85:      public static void main(String[] args) throws Throwable {

Quick drive-by comment: The indentation is off, also below.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24192#discussion_r2010043449