RFR: 8366890: C2: Split through phi printing with TraceLoopOpts misses line break
Christian Hagedorn
chagedorn at openjdk.org
Fri Sep 5 15:29:16 UTC 2025
On Thu, 4 Sep 2025 13:23:24 GMT, Roberto Castañeda Lozano <rcastanedalo 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
>
> Looks good, and trivial.
Thanks @robcasloz, @mhaessig and @eme64 for your reviews! And no worries @mhaessig, was easy to overlook :-)
-------------
PR Comment: https://git.openjdk.org/jdk/pull/27092#issuecomment-3258771939
More information about the hotspot-compiler-dev
mailing list