Integrated: 8330469: C2: Remove or change "PrintOpto && VerifyLoopOptimizations" as printing code condition
Saranya Natarajan
duke at openjdk.org
Thu Mar 13 11:05:12 UTC 2025
On Mon, 10 Mar 2025 10:34:19 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)
This pull request has now been integrated.
Changeset: 9c003314
Author: Saranya Natarajan <saranya.natarajan at oracle.com>
Committer: Roberto Castañeda Lozano <rcastanedalo at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/9c00331465fe83e491f6dd1e6df4df1fb790f2fc
Stats: 38 lines in 3 files changed: 0 ins; 34 del; 4 mod
8330469: C2: Remove or change "PrintOpto && VerifyLoopOptimizations" as printing code condition
Reviewed-by: chagedorn, rcastanedalo
-------------
PR: https://git.openjdk.org/jdk/pull/23959
More information about the hotspot-compiler-dev
mailing list