RFR: 8341977: Replace predicate walking and cloning code for Loop Peeling with a predicate visitor [v3]

[Emanuel Peter]

On Mon, 28 Oct 2024 13:14:29 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Christian Hagedorn has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Review Emanuel
>
> src/hotspot/share/opto/loopTransform.cpp line 1991:
> 
>> 1989:   predicate_iterator.for_each(create_assertion_predicates_for_loop);
>> 1990:   if (create_assertion_predicates_for_loop.has_created_predicates()) {
>> 1991:     IfTrueNode* last_created_predicate_success_proj = create_assertion_predicates_for_loop.last_created_success_proj();
> 
> Suggestion:
> 
>   CreateAssertionPredicatesVisitor create_assertion_predicates_visitor(init, stride, target_loop_entry, this,
>                                                                         _node_in_loop_body);
>   Node* source_loop_entry = source_loop_head->skip_strip_mined()->in(LoopNode::EntryControl);
>   PredicateIterator predicate_iterator(source_loop_entry);
>   predicate_iterator.for_each(create_assertion_predicates_visitor);
>   if (create_assertion_predicates_visitor.has_created_predicates()) {
>     IfTrueNode* last_created_predicate_success_proj = create_assertion_predicates_for_loop.last_created_success_proj();

I think this would complete the renaming

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21679#discussion_r1819038342