RFR: 8269820: C2 PhaseIdealLoop::do_unroll get wrong opaque node [v2]

Thu Dec 1 16:30:30 UTC 2022

On Wed, 30 Nov 2022 10:51:55 GMT, Roland Westrelin <roland at openjdk.org> wrote:

>> I mean we never assign control edge to `Bool` and `Cmp` nodes - they depend only on their inputs. At least I don't know about it.
>
> `set_ctrl()` doesn't change the control input of the nodes, right? It only updates the current loop opts pass's table of controls and all data nodes are in that table. I'm confused by what could be wrong here.

You are right, I forgot that this function only update site table. Good.

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

PR: https://git.openjdk.org/jdk/pull/11391