[jdk17] Withdrawn: 8269820: C2 PhaseIdealLoop::do_unroll get wrong opaque node

[Kevin Rushforth]

On Mon, 19 Jul 2021 14:13:31 GMT, Hui Shi <hshi at openjdk.org> wrote:

> More discussion in previous PR(https://github.com/openjdk/jdk17/pull/208) is confilict with fix for JDK-8269752.
> 
> Opaque1 in main loop entry is expected only used in compare node's second input. However split if might clone opaque1 and replace original node with phi of opaque1 node. This causes assertion in CountedLoopNode::is_canonical_loop_entry now.
> 
> Fix is in BoolNode::Ideal, avoiding switching compare node's input when second input is phi of opaque1.
> 
> Test: Linux X64 tier1/2/3 release/fastdebug no regression.

This pull request has been closed without being integrated.

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

PR: https://git.openjdk.java.net/jdk17/pull/255