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

Saranya Natarajan duke at openjdk.org
Tue Mar 11 11:10:10 UTC 2025


> **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()

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/23959/files
  - new: https://git.openjdk.org/jdk/pull/23959/files/91ecc519..b7216ca6

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=23959&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=23959&range=00-01

  Stats: 30 lines in 3 files changed: 0 ins; 30 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/23959.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/23959/head:pull/23959

PR: https://git.openjdk.org/jdk/pull/23959


More information about the hotspot-compiler-dev mailing list