RFR: 8251271: C2: Compile::_for_igvn list is corrupted after RenumberLiveNodes [v2]

[Nhat Nguyen]

On Sat, 17 Oct 2020 21:38:35 GMT, Xin Liu <xliu at openjdk.org> wrote:

>> 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

Thank you for the suggestion. I have updated the PR.

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

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