RFR: 8352696: JFR: assert(false): EA: missing memory path [v5]
Aleksey Shipilev
shade at openjdk.org
Tue Mar 25 08:01:20 UTC 2025
On Mon, 24 Mar 2025 19:23: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:
>
> simplified test
Marked as reviewed by shade (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/24192#pullrequestreview-2712743344
More information about the hotspot-compiler-dev
mailing list