RFR: 8367690: C2: Unneeded branch in reduce_phi
Christian Hagedorn
chagedorn at openjdk.org
Tue Oct 21 09:38:14 UTC 2025
On Thu, 16 Oct 2025 14:49:02 GMT, Anton Seoane Ampudia <aseoane at openjdk.org> wrote:
> This PR carries out a minor cleanup found in the Phi reduction code.
>
> The combination of the branch and assert is redundant as the assert will always trigger. We can either remove the `else if` branch or change the assert for a `ShouldNotReachHere`.
>
> I have gone for the former option here.
>
> **Testing:** passes tiers 1-3
Looks good!
-------------
Marked as reviewed by chagedorn (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/27849#pullrequestreview-3359711380
More information about the hotspot-compiler-dev
mailing list