RFR: 8330469: C2: Remove or change "PrintOpto && VerifyLoopOptimizations" as printing code condition [v3]
Saranya Natarajan
duke at openjdk.org
Wed Mar 12 10:51:43 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 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
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/23959/files
- new: https://git.openjdk.org/jdk/pull/23959/files/b7216ca6..3ee2e887
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=23959&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=23959&range=01-02
Stats: 48784 lines in 726 files changed: 24534 ins; 15325 del; 8925 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