RFR: 8251271: C2: Compile::_for_igvn list is corrupted after RenumberLiveNodes

Sat Oct 17 21:41:09 UTC 2020

On Sat, 17 Oct 2020 13:54:20 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:

>> I'm following up on this thread http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2020-September/039910.html
>> I'm clearing `for_igvn` before restoring as suggested by Xin
>
> src/hotspot/share/opto/compile.cpp line 2115:
> 
>> 2113:     // Clear the for_igvn list because it may have irrelevant nodes
>> 2114:     // from the previous PhaseRenumberLive run.
>> 2115:     save_for_igvn->clear();
> 
> Since `PhaseRenumberLive::PhaseRenumberLive` moves nodes from `for_igvn()` to `new_worklist`, does it make more sense
> to drain original `for_igvn()` worklist there?

+1

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

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