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

[Christian Hagedorn]

On Tue, 11 Mar 2025 11:10:10 GMT, Saranya Natarajan <duke at openjdk.org> wrote:

>> **Issue:** There are currently 9 occurrences where we guard printing code with PrintOpto && VerifyLoopOptimizations. This flag combo is never really used in practice. 
>> 
>> **Solution**: I analysed the 9 occurrence. In cases, where the flag `PrintOpto && VerifyLoopOptimizations` was followed by flag `TraceLoopOpts` with `else if` or  `|| operator` I removed the former flag. In other cases, where `PrintOpto && VerifyLoopOptimizations` was the only flag, I was replaced with `TraceLoopOpts`. 
>> 
>> **Test Result**: Link to [GitHub Action](https://github.com/sarannat/jdk/actions/runs/13723071055) run on commit [91ecc51](https://github.com/sarannat/jdk/commit/91ecc5190ce31da94bded4de210136f337286e69)
>
> Saranya Natarajan has updated the pull request incrementally with one additional commit since the last revision:
> 
>   JDK-8330469: Addressing review comments by removing TraceLoopOpts and some dump()

Thanks for the update, looks good!

Can you file an RFE for `TraceSplitIf` and link it to this RFE? Then we can keep track of this :-)

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

Marked as reviewed by chagedorn (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/23959#pullrequestreview-2675176488