RFR: 8366890: C2: Split through phi printing with TraceLoopOpts misses line break

Manuel Hässig mhaessig at openjdk.org
Thu Sep 4 13:30:42 UTC 2025


On Thu, 4 Sep 2025 12:44:43 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

> [JDK-8356176](https://bugs.openjdk.org/browse/JDK-8356176) added new printing code for `TraceLoopOpts` when splitting nodes through a phi but missed a line break. This will result in:
> 
> Split 974 CmpI through 1465 Phi in 953 RegionSplit 474 Bool through 1468 Phi in 953 RegionSplit-If
> 
> instead of
> 
> Split 974 CmpI through 1465 Phi in 953 RegionSplit 474 Bool through 1468 Phi in 953 Region
> Split-If
> 
> This patch fixes this.
> 
> Thanks,
> Christian

Thank you for fixing my silly mistake, @chhagedorn! Looks good to me as well.

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

Marked as reviewed by mhaessig (Committer).

PR Review: https://git.openjdk.org/jdk/pull/27092#pullrequestreview-3185304718


More information about the hotspot-compiler-dev mailing list