RFR: 8271954: C2: assert(false) failed: Bad graph detected in build_loop_late [v2]

[Christian Hagedorn]

On Tue, 24 Aug 2021 15:02:31 GMT, Roland Westrelin <roland at openjdk.org> wrote:

>> I'm not sure because `old_new` is shared in loopopts. I was concerned that I find old mappings here which I tried to exclude.
>
> Shouldn't it be reset between use i.e.:
> memset( worklist.adr(), 0, worklist.Size()*sizeof(Node*) );
> ?

Yes, I was mainly worried about visiting nodes from the current unswitching phase. But I think that cannot happen. I'll change it into an assert.

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

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