RFR: 8330469: C2: Remove or change "PrintOpto && VerifyLoopOptimizations" as printing code condition [v3]

[Saranya Natarajan]

On Mon, 10 Mar 2025 12:00:24 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Saranya Natarajan has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
>> 
>>  - Merge branch 'master' into JDK-8330469
>>  - JDK-8330469: Addressing review comments by removing TraceLoopOpts and some dump()
>>  - 8330469 : Removed or replaced (PrintOpto && VerifyLoopOptimizations) with TraceLoopOpts
>
> src/hotspot/share/opto/split_if.cpp line 143:
> 
>> 141:     tty->print("Cloning up: ");
>> 142:     n->dump();
>> 143:   }
> 
> Same here with this print and the other places in `clone_cmp_down()`: I think it's too verbose for `TraceLoopOpts`. 
> 
> Since Split-If is quite complex, I think it would make sense to add a "TraceSplitIf" flag to get more information about the optimization. It's probably out of scope of this bug, so we could do that in a separate RFE. For this PR, I suggest to just drop these printings and link this PR to the "TraceSplitIf" RFE in order to restore/update/improve these.

Thank you for the review. I have created RFE [JDK-8351847](https://bugs.openjdk.org/browse/JDK-8351847) for the "TraceSplitIf"

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23959#discussion_r1991419982