Integrated: 8256385: C2: fatal error: modified node is not on IGVN._worklist

[Tobias Hartmann]

On Tue, 17 Nov 2020 08:49:32 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

> `PhaseIdealLoop::find_safepoint` creates a temporary MergeMemNode that is not removed if we bail out from the optimization early (see `return NULL` statements). The fix is to simply add the MergeMem to the worklist to make sure it is always reclaimed by IGVN.
> 
> Interestingly this code path was not triggered by any of our tests but only with a test case generated by the Java Fuzzer. I've added a simplified version of that test case.
> 
> Thanks,
> Tobias

This pull request has now been integrated.

Changeset: f504f419
Author:    Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/f504f419
Stats:     55 lines in 2 files changed: 55 ins; 0 del; 0 mod

8256385: C2: fatal error: modified node is not on IGVN._worklist

Reviewed-by: chagedorn, roland

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

PR: https://git.openjdk.java.net/jdk/pull/1252