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