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

Tue Nov 17 09:53:05 UTC 2020

On Tue, 17 Nov 2020 09:44:54 GMT, Roland Westrelin <roland 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
>
> Marked as reviewed by roland (Reviewer).

Sounds good, thanks for the review!

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

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