RFR: 8350756: C2 SuperWord Multiversioning: remove useless slow loop when the fast loop disappears [v2]

[Christian Hagedorn]

On Tue, 4 Mar 2025 10:21:59 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

> Then we can just clear the relevant arrays on start of each loop-opts phase. That way we do not have to remove individual nodes, and we can still reuse the allocated memory.

Yes, that would be ideal. Might be worth to further discuss this and what the impact would be.

Anyway, I don't think we need to settle this now. Since zero trip guard opaque nodes already introduced a second way of doing it and there is no clear better way, I think we can use one or the other solution (your call which one) for now and visit this later again to unify the solution - hopefully with a new LoopOpts class :-) Is it worth to file an RFE now to keep track of that?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23865#discussion_r1979158705