RFR: 8351392: C2 crash: failed: Expected Bool, but got OpaqueMultiversioning [v2]

[Emanuel Peter]

On Fri, 7 Mar 2025 11:40:14 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> That is essencially what we I already do. If we find a **useful** node, then we just hit the assert further down.
>> 
>> But if you want then I can refactor it explicitly.
>
> It might be more explicit with a separate assert that an `OpaqueMultiversioning` should be useless. And we can avoid the `is_useless()` check in product - but of course that should not make much of a difference performance wise :-)

Ok, I refactored it :)

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23943#discussion_r1984963252