Integrated: 8367690: C2: Unneeded branch in reduce_phi
    Anton Seoane Ampudia 
    aseoane at openjdk.org
       
    Wed Oct 22 09:11:59 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
This pull request has now been integrated.
Changeset: bdfd5e84
Author:    Anton Seoane Ampudia <aseoane at openjdk.org>
Committer: Roberto Castañeda Lozano <rcastanedalo at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/bdfd5e843a7d3db50edf4375e50449b0ce528f8a
Stats:     3 lines in 1 file changed: 1 ins; 2 del; 0 mod
8367690: C2: Unneeded branch in reduce_phi
Reviewed-by: rcastanedalo, chagedorn
-------------
PR: https://git.openjdk.org/jdk/pull/27849
    
    
More information about the hotspot-compiler-dev
mailing list