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

[Roland Westrelin]

On Fri, 27 Aug 2021 09:09:36 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> It would seem a single variable that carries the tri state would be the clearest option. That would require an enum though so not sure if it's worth it. Anyway, I'm good with the change as it is. I only found the 2 boolean variables a bit confusing.
>
> I agree that this is confusing. I think your suggestion with an enum is the most clean way and also improves the readability. I'll change that.

Looks good.

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

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