https://gcc.gnu.org/g:b876a7fc805b9c7999c933a943eeb3b833780256
commit r16-1109-gb876a7fc805b9c7999c933a943eeb3b833780256
Author: Jakub Jelinek <ja...@redhat.com>
Date: Wed Jun 4 17:22:58 2025 +0200
ranger: Some parameter formatting fixes
When reading the code, I've noticed various function definitions
with misaligned parameters, they should IMHO always align below the first
character after opening ( and in most cases they do, but in some
cases they were indented more or less. Perhaps the functions changed
name or something.
2025-06-04 Jakub Jelinek <ja...@redhat.com>
* range-op-float.cc (range_operator::fold_range,
range_operator::op1_range, range_operator::op2_range,
range_operator::lhs_op1_relation, range_operator::lhs_op2_relation,
operator_equal::op1_range, foperator_unordered_gt::op1_range): Fix
up parameter indentation.
* range-op.cc (range_operator::fold_range,
range_operator::op1_range,
range_operator::op1_op2_relation_effect,
range_operator::update_bitmask, plus_minus_ranges,
operator_bitwise_and::lhs_op1_relation): Likewise.
Diff:
---
gcc/range-op-float.cc | 82 +++++++++++++++++++++++++--------------------------
gcc/range-op.cc | 27 +++++++++--------
2 files changed, 56 insertions(+), 53 deletions(-)
diff --git a/gcc/range-op-float.cc b/gcc/range-op-float.cc
index 1543b94b878e..ea344a4d043f 100644
--- a/gcc/range-op-float.cc
+++ b/gcc/range-op-float.cc
@@ -51,8 +51,8 @@ along with GCC; see the file COPYING3. If not see
bool
range_operator::fold_range (frange &r, tree type,
- const frange &op1, const frange &op2,
- relation_trio trio) const
+ const frange &op1, const frange &op2,
+ relation_trio trio) const
{
if (empty_range_varying (r, type, op1, op2))
return true;
@@ -112,20 +112,20 @@ range_operator::rv_fold (frange &r, tree type,
bool
range_operator::fold_range (irange &r ATTRIBUTE_UNUSED,
- tree type ATTRIBUTE_UNUSED,
- const frange &lh ATTRIBUTE_UNUSED,
- const irange &rh ATTRIBUTE_UNUSED,
- relation_trio) const
+ tree type ATTRIBUTE_UNUSED,
+ const frange &lh ATTRIBUTE_UNUSED,
+ const irange &rh ATTRIBUTE_UNUSED,
+ relation_trio) const
{
return false;
}
bool
range_operator::fold_range (irange &r ATTRIBUTE_UNUSED,
- tree type ATTRIBUTE_UNUSED,
- const frange &lh ATTRIBUTE_UNUSED,
- const frange &rh ATTRIBUTE_UNUSED,
- relation_trio) const
+ tree type ATTRIBUTE_UNUSED,
+ const frange &lh ATTRIBUTE_UNUSED,
+ const frange &rh ATTRIBUTE_UNUSED,
+ relation_trio) const
{
return false;
}
@@ -142,10 +142,10 @@ range_operator::fold_range (frange &r ATTRIBUTE_UNUSED,
bool
range_operator::op1_range (frange &r ATTRIBUTE_UNUSED,
- tree type ATTRIBUTE_UNUSED,
- const frange &lhs ATTRIBUTE_UNUSED,
- const frange &op2 ATTRIBUTE_UNUSED,
- relation_trio) const
+ tree type ATTRIBUTE_UNUSED,
+ const frange &lhs ATTRIBUTE_UNUSED,
+ const frange &op2 ATTRIBUTE_UNUSED,
+ relation_trio) const
{
return false;
}
@@ -162,56 +162,56 @@ range_operator::op1_range (frange &r ATTRIBUTE_UNUSED,
bool
range_operator::op2_range (frange &r ATTRIBUTE_UNUSED,
- tree type ATTRIBUTE_UNUSED,
- const frange &lhs ATTRIBUTE_UNUSED,
- const frange &op1 ATTRIBUTE_UNUSED,
- relation_trio) const
+ tree type ATTRIBUTE_UNUSED,
+ const frange &lhs ATTRIBUTE_UNUSED,
+ const frange &op1 ATTRIBUTE_UNUSED,
+ relation_trio) const
{
return false;
}
bool
range_operator::op2_range (frange &r ATTRIBUTE_UNUSED,
- tree type ATTRIBUTE_UNUSED,
- const irange &lhs ATTRIBUTE_UNUSED,
- const frange &op1 ATTRIBUTE_UNUSED,
- relation_trio) const
+ tree type ATTRIBUTE_UNUSED,
+ const irange &lhs ATTRIBUTE_UNUSED,
+ const frange &op1 ATTRIBUTE_UNUSED,
+ relation_trio) const
{
return false;
}
relation_kind
range_operator::lhs_op1_relation (const frange &lhs ATTRIBUTE_UNUSED,
- const frange &op1 ATTRIBUTE_UNUSED,
- const frange &op2 ATTRIBUTE_UNUSED,
- relation_kind) const
+ const frange &op1 ATTRIBUTE_UNUSED,
+ const frange &op2 ATTRIBUTE_UNUSED,
+ relation_kind) const
{
return VREL_VARYING;
}
relation_kind
range_operator::lhs_op1_relation (const irange &lhs ATTRIBUTE_UNUSED,
- const frange &op1 ATTRIBUTE_UNUSED,
- const frange &op2 ATTRIBUTE_UNUSED,
- relation_kind) const
+ const frange &op1 ATTRIBUTE_UNUSED,
+ const frange &op2 ATTRIBUTE_UNUSED,
+ relation_kind) const
{
return VREL_VARYING;
}
relation_kind
range_operator::lhs_op2_relation (const irange &lhs ATTRIBUTE_UNUSED,
- const frange &op1 ATTRIBUTE_UNUSED,
- const frange &op2 ATTRIBUTE_UNUSED,
- relation_kind) const
+ const frange &op1 ATTRIBUTE_UNUSED,
+ const frange &op2 ATTRIBUTE_UNUSED,
+ relation_kind) const
{
return VREL_VARYING;
}
relation_kind
range_operator::lhs_op2_relation (const frange &lhs ATTRIBUTE_UNUSED,
- const frange &op1 ATTRIBUTE_UNUSED,
- const frange &op2 ATTRIBUTE_UNUSED,
- relation_kind) const
+ const frange &op1 ATTRIBUTE_UNUSED,
+ const frange &op2 ATTRIBUTE_UNUSED,
+ relation_kind) const
{
return VREL_VARYING;
}
@@ -675,9 +675,9 @@ operator_equal::fold_range (irange &r, tree type,
bool
operator_equal::op1_range (frange &r, tree type,
- const irange &lhs,
- const frange &op2,
- relation_trio trio) const
+ const irange &lhs,
+ const frange &op2,
+ relation_trio trio) const
{
relation_kind rel = trio.op1_op2 ();
switch (get_bool_state (r, lhs, type))
@@ -1871,10 +1871,10 @@ public:
bool
foperator_unordered_gt::op1_range (frange &r,
- tree type,
- const irange &lhs,
- const frange &op2,
- relation_trio) const
+ tree type,
+ const irange &lhs,
+ const frange &op2,
+ relation_trio) const
{
switch (get_bool_state (r, lhs, type))
{
diff --git a/gcc/range-op.cc b/gcc/range-op.cc
index e2b9c82bc7b7..04128ee2dde5 100644
--- a/gcc/range-op.cc
+++ b/gcc/range-op.cc
@@ -778,21 +778,21 @@ range_operator::fold_range (irange &r, tree type,
bool
range_operator::fold_range (frange &, tree, const irange &,
- const frange &, relation_trio) const
+ const frange &, relation_trio) const
{
return false;
}
bool
range_operator::op1_range (irange &, tree, const frange &,
- const frange &, relation_trio) const
+ const frange &, relation_trio) const
{
return false;
}
bool
range_operator::op1_range (frange &, tree, const irange &,
- const irange &, relation_trio) const
+ const irange &, relation_trio) const
{
return false;
}
@@ -855,10 +855,13 @@ range_operator::op1_op2_relation (const irange &lhs
ATTRIBUTE_UNUSED,
bool
range_operator::op1_op2_relation_effect (irange &lhs_range ATTRIBUTE_UNUSED,
- tree type ATTRIBUTE_UNUSED,
- const irange &op1_range ATTRIBUTE_UNUSED,
- const irange &op2_range ATTRIBUTE_UNUSED,
- relation_kind rel ATTRIBUTE_UNUSED) const
+ tree type ATTRIBUTE_UNUSED,
+ const irange &op1_range
+ ATTRIBUTE_UNUSED,
+ const irange &op2_range
+ ATTRIBUTE_UNUSED,
+ relation_kind rel
+ ATTRIBUTE_UNUSED) const
{
return false;
}
@@ -874,7 +877,7 @@ range_operator::overflow_free_p (const irange &, const
irange &,
void
range_operator::update_bitmask (irange &, const irange &,
- const irange &) const
+ const irange &) const
{
}
@@ -1815,7 +1818,7 @@ operator_plus::wi_fold (irange &r, tree type,
static relation_kind
plus_minus_ranges (irange &r_ov, irange &r_normal, const irange &offset,
- bool add_p)
+ bool add_p)
{
relation_kind kind = VREL_VARYING;
// For now, only deal with constant adds. This could be extended to ranges
@@ -3349,9 +3352,9 @@ wi_optimize_signed_bitwise_op (irange &r, tree type,
relation_kind
operator_bitwise_and::lhs_op1_relation (const irange &lhs,
- const irange &op1,
- const irange &op2,
- relation_kind) const
+ const irange &op1,
+ const irange &op2,
+ relation_kind) const
{
if (lhs.undefined_p () || op1.undefined_p () || op2.undefined_p ())
return VREL_VARYING;
