RFR: 8347018: C2: Insertion of Assertion Predicates ignores the effects of PhaseIdealLoop::clone_up_backedge_goo() [v2]

Fri Jan 17 10:04:45 UTC 2025

On Fri, 17 Jan 2025 09:56:39 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

> > Regardless of the observed assertion failure, I think the cloned nodes only belonging to the main loop should not be pinned before the pre loop
> 
> That sounds like a good argument. I will review the code now ;)
> 
> Thanks for all the extra explanations 😊

Sure, you're welcome! :-) 

Sounds good, thanks a lot!

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

PR Comment: https://git.openjdk.org/jdk/pull/23071#issuecomment-2597866044