[gcc(refs/users/aoliva/heads/testme)] drop redundant ifcombine_ifandif parm

Alexandre Oliva aoliva@gcc.gnu.org
Thu Oct 24 08:26:40 GMT 2024


https://gcc.gnu.org/g:ebc7c0f5cc561a8dd26ab36dec48c2c13b6a9c24

commit ebc7c0f5cc561a8dd26ab36dec48c2c13b6a9c24
Author: Alexandre Oliva <oliva@gnu.org>
Date:   Thu Oct 24 05:25:24 2024 -0300

    drop redundant ifcombine_ifandif parm

Diff:
---
 gcc/tree-ssa-ifcombine.cc | 17 ++++++-----------
 1 file changed, 6 insertions(+), 11 deletions(-)

diff --git a/gcc/tree-ssa-ifcombine.cc b/gcc/tree-ssa-ifcombine.cc
index ed20a231951a..eae32e1e1999 100644
--- a/gcc/tree-ssa-ifcombine.cc
+++ b/gcc/tree-ssa-ifcombine.cc
@@ -401,13 +401,12 @@ update_profile_after_ifcombine (basic_block inner_cond_bb,
 
 /* If-convert on a and pattern with a common else block.  The inner
    if is specified by its INNER_COND_BB, the outer by OUTER_COND_BB.
-   inner_inv, outer_inv and result_inv indicate whether the conditions
-   are inverted.
+   inner_inv, outer_inv indicate whether the conditions are inverted.
    Returns true if the edges to the common else basic-block were merged.  */
 
 static bool
 ifcombine_ifandif (basic_block inner_cond_bb, bool inner_inv,
-		   basic_block outer_cond_bb, bool outer_inv, bool result_inv)
+		   basic_block outer_cond_bb, bool outer_inv)
 {
   gimple_stmt_iterator gsi;
   tree name1, name2, bit1, bit2, bits1, bits2;
@@ -693,8 +692,7 @@ tree_ssa_ifcombine_bb_1 (basic_block inner_cond_bb, basic_block outer_cond_bb,
 	   <else_bb>
 	     ...
        */
-      return ifcombine_ifandif (inner_cond_bb, false, outer_cond_bb, false,
-				false);
+      return ifcombine_ifandif (inner_cond_bb, false, outer_cond_bb, false);
     }
 
   /* And a version where the outer condition is negated.  */
@@ -711,8 +709,7 @@ tree_ssa_ifcombine_bb_1 (basic_block inner_cond_bb, basic_block outer_cond_bb,
 	   <else_bb>
 	     ...
        */
-      return ifcombine_ifandif (inner_cond_bb, false, outer_cond_bb, true,
-				false);
+      return ifcombine_ifandif (inner_cond_bb, false, outer_cond_bb, true);
     }
 
   /* The || form is characterized by a common then_bb with the
@@ -731,8 +728,7 @@ tree_ssa_ifcombine_bb_1 (basic_block inner_cond_bb, basic_block outer_cond_bb,
 	   <then_bb>
 	     ...
        */
-      return ifcombine_ifandif (inner_cond_bb, true, outer_cond_bb, true,
-				true);
+      return ifcombine_ifandif (inner_cond_bb, true, outer_cond_bb, true);
     }
 
   /* And a version where the outer condition is negated.  */
@@ -748,8 +744,7 @@ tree_ssa_ifcombine_bb_1 (basic_block inner_cond_bb, basic_block outer_cond_bb,
 	   <then_bb>
 	     ...
        */
-      return ifcombine_ifandif (inner_cond_bb, true, outer_cond_bb, false,
-				true);
+      return ifcombine_ifandif (inner_cond_bb, true, outer_cond_bb, false);
     }
 
   return false;


More information about the Gcc-cvs mailing list