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

Vladimir Kozlov kvn at openjdk.org
Mon Mar 24 17:16:22 UTC 2025


On Mon, 24 Mar 2025 12:39:56 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
>
> Markus Grönlund has updated the pull request incrementally with one additional commit since the last revision:
> 
>   fold

Looks good.

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

Marked as reviewed by kvn (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/24192#pullrequestreview-2711139109


More information about the hotspot-compiler-dev mailing list