Skip to content

Commit

Permalink
Fix up duplicated words mostly in comments, part 2
Browse files Browse the repository at this point in the history
Another patch from eyeballing
git grep -v 'long long\|optab optab\|template template\|double double' | grep ' \([a-zA-Z]\+\) \1 '
output, this time in gcc/ subdirectory.

2024-04-09  Jakub Jelinek  <[email protected]>

gcc/
	* expr.cc (convert_mode_scalar): Fix duplicated words in comment;
	into into -> it into.
	* function.h (function::cond_uids): Fix duplicated words in comment;
	same same -> same.
	* config/riscv/riscv-vector-costs.cc
	(costs::adjust_vect_cost_per_loop): Fix duplicated words in comment;
	model model -> model.
	* config/riscv/riscv-vector-builtins-shapes.cc (build_base): Fix
	duplicated words in comment; for for -> for.
	* config/riscv/riscv-avlprop.cc (pass_avlprop::execute): Fix
	duplicated words in comment; more more -> more.
	* config/aarch64/driver-aarch64.cc (host_detect_local_cpu): Fix
	duplicated words in comment; be be -> be.
	* tree-profile.cc (masking_vectors): Fix duplicated words in comment;
	has has -> has, the the -> the.
	* value-range.cc (irange::set_range_from_bitmask): Fix duplicated
	words in comment; the the -> the.
	* gcov.cc (add_condition_counts): Fix duplicated words in comment;
	to to -> to.
	* vr-values.cc (get_scev_info): Fix duplicated words in comment;
	the the -> to the.
	* tree-vrp.cc (fully_replaceable): Fix duplicated words in comment;
	by by -> by.
	* mode-switching.cc (single_succ_confluence_n): Fix duplicated words
	in comment; the the -> the.
	* tree-ssa-phiopt.cc (value_replacement): Fix duplicated words in
	comment; can can -> we can.
	* gimple-range-phi.cc (phi_analyzer::process_phi): Fix duplicated words
	in comment; it it -> it is.
	* tree-ssa-sccvn.cc (visit_phi): Fix duplicated words in comment;
	to to -> to.
	* rtl-ssa/accesses.h (use_info::next_debug_insn_use): Fix duplicated
	words in comment; if if -> if.
	* doc/options.texi (InverseMask): Fix duplicated words; and and -> and.
	Change take to takes.
	* doc/invoke.texi (fanalyzer-undo-inlining): Fix duplicated words;
	be be -> be.
	(-minline-memops-threshold): Likewise.
gcc/analyzer/
	* analyzer.opt (Wanalyzer-undefined-behavior-strtok): Fix duplicated
	words; in in -> in.
	* program-state.cc (sm_state_map::replay_call_summary): Fix duplicated
	words in comment; to to -> to.
	(program_state::replay_call_summary): Likewise.
	* region-model.cc (region_model::replay_call_summary): Likewise.
gcc/c/
	* c-decl.cc (previous_tag): Fix duplicated words in comment; the the
	-> the.
	(diagnose_mismatched_decls): Fix duplicated words in comment;
	about about -> about.
gcc/cp/
	* constexpr.cc (build_new_constexpr_heap_type): Fix duplicated words
	in comment; is is -> is.
	* cp-tree.def (CO_RETURN_EXPR): Fix duplicated words in comment;
	for for -> for.
	* parser.cc (fixup_blocks_walker): Fix duplicated words in comment;
	is is -> is.
	* semantics.cc (fixup_template_type): Fix duplicated words in comment;
	for for -> for.
	(finish_omp_for): Fix duplicated words in comment; the the -> the.
	* pt.cc (more_specialized_fn): Fix duplicated words in comment;
	think think -> think.
	(type_targs_deducible_from): Fix duplicated words in comment; the the
	-> the.
gcc/jit/
	* docs/topics/expressions.rst (Constructor expressions): Fix
	duplicated words; have have -> have.
  • Loading branch information
jakubjelinek committed Apr 9, 2024
1 parent 7dd1f9d commit 64aa48c
Show file tree
Hide file tree
Showing 28 changed files with 36 additions and 36 deletions.
2 changes: 1 addition & 1 deletion gcc/analyzer/analyzer.opt
Original file line number Diff line number Diff line change
Expand Up @@ -224,7 +224,7 @@ Warn about code paths in which an unsanitized value is used as a size.

Wanalyzer-undefined-behavior-strtok
Common Var(warn_analyzer_undefined_behavior_strtok) Init(1) Warning
Warn about code paths in in which a call is made to strtok with undefined behavior.
Warn about code paths in which a call is made to strtok with undefined behavior.

Wanalyzer-use-after-free
Common Var(warn_analyzer_use_after_free) Init(1) Warning
Expand Down
4 changes: 2 additions & 2 deletions gcc/analyzer/program-state.cc
Original file line number Diff line number Diff line change
Expand Up @@ -877,7 +877,7 @@ program_state::program_state (const extrinsic_state &ext_state)
}
}

/* Attempt to to use R to replay SUMMARY into this object.
/* Attempt to use R to replay SUMMARY into this object.
Return true if it is possible. */

bool
Expand Down Expand Up @@ -1625,7 +1625,7 @@ program_state::detect_leaks (const program_state &src_state,
dest_state.m_region_model->unset_dynamic_extents (reg);
}

/* Attempt to to use R to replay SUMMARY into this object.
/* Attempt to use R to replay SUMMARY into this object.
Return true if it is possible. */

bool
Expand Down
2 changes: 1 addition & 1 deletion gcc/analyzer/region-model.cc
Original file line number Diff line number Diff line change
Expand Up @@ -5623,7 +5623,7 @@ region_model::update_for_return_superedge (const return_superedge &return_edge,
update_for_return_gcall (call_stmt, ctxt);
}

/* Attempt to to use R to replay SUMMARY into this object.
/* Attempt to use R to replay SUMMARY into this object.
Return true if it is possible. */

bool
Expand Down
4 changes: 2 additions & 2 deletions gcc/c/c-decl.cc
Original file line number Diff line number Diff line change
Expand Up @@ -2069,7 +2069,7 @@ locate_old_decl (tree decl)


/* Helper function. For a tagged type, it finds the declaration
for a visible tag declared in the the same scope if such a
for a visible tag declared in the same scope if such a
declaration exists. */
static tree
previous_tag (tree type)
Expand Down Expand Up @@ -2316,7 +2316,7 @@ diagnose_mismatched_decls (tree newdecl, tree olddecl,
(C23 6.7.2.2/5), but may pose portability problems. */
else if (enum_and_int_p
&& TREE_CODE (newdecl) != TYPE_DECL
/* Don't warn about about acc_on_device built-in redeclaration,
/* Don't warn about acc_on_device built-in redeclaration,
the built-in is declared with int rather than enum because
the enum isn't intrinsic. */
&& !(TREE_CODE (olddecl) == FUNCTION_DECL
Expand Down
2 changes: 1 addition & 1 deletion gcc/config/aarch64/driver-aarch64.cc
Original file line number Diff line number Diff line change
Expand Up @@ -452,7 +452,7 @@ host_detect_local_cpu (int argc, const char **argv)
if (tune)
return res;

/* Add any features that should be be present, but can't be verified using
/* Add any features that should be present, but can't be verified using
the /proc/cpuinfo "Features" list. */
extension_flags |= unchecked_extension_flags & default_flags;

Expand Down
2 changes: 1 addition & 1 deletion gcc/config/riscv/riscv-avlprop.cc
Original file line number Diff line number Diff line change
Expand Up @@ -458,7 +458,7 @@ pass_avlprop::execute (function *fn)

/* TODO: We only do AVL propagation for VLMAX AVL with tail
agnostic policy since we have missed-LEN information partial
autovectorization. We could add more more AVL propagation
autovectorization. We could add more AVL propagation
for intrinsic codes in the future. */
if (vlmax_ta_p (insn->rtl ()))
m_candidates.safe_push (std::make_pair (AVLPROP_VLMAX_TA, insn));
Expand Down
2 changes: 1 addition & 1 deletion gcc/config/riscv/riscv-vector-builtins-shapes.cc
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ build_all (function_builder &b, const function_group_info &group)

#define BASE_NAME_MAX_LEN 16

/* Base class for for build. */
/* Base class for build. */
struct build_base : public function_shape
{
void build (function_builder &b,
Expand Down
2 changes: 1 addition & 1 deletion gcc/config/riscv/riscv-vector-costs.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1170,7 +1170,7 @@ costs::add_stmt_cost (int count, vect_cost_for_stmt kind,

/* For some target specific vectorization cost which can't be handled per stmt,
we check the requisite conditions and adjust the vectorization cost
accordingly if satisfied. One typical example is to model model and adjust
accordingly if satisfied. One typical example is to model and adjust
loop_len cost for known_lt (NITERS, VF). */

void
Expand Down
2 changes: 1 addition & 1 deletion gcc/cp/constexpr.cc
Original file line number Diff line number Diff line change
Expand Up @@ -7224,7 +7224,7 @@ maybe_warn_about_constant_value (location_t loc, tree decl)
in bytes. If COOKIE_SIZE is NULL, return array type
ELT_TYPE[FULL_SIZE / sizeof(ELT_TYPE)], otherwise return
struct { size_t[COOKIE_SIZE/sizeof(size_t)]; ELT_TYPE[N]; }
where N is is computed such that the size of the struct fits into FULL_SIZE.
where N is computed such that the size of the struct fits into FULL_SIZE.
If ARG_SIZE is non-NULL, it is the first argument to the new operator.
It should be passed if ELT_TYPE is zero sized type in which case FULL_SIZE
will be also 0 and so it is not possible to determine the actual array
Expand Down
2 changes: 1 addition & 1 deletion gcc/cp/cp-tree.def
Original file line number Diff line number Diff line change
Expand Up @@ -567,7 +567,7 @@ DEFTREECODE (CO_YIELD_EXPR, "co_yield", tcc_expression, 2)
/* The co_return expression is used to support coroutines.

Op0 is the original expr, can be void (for use in diagnostics)
Op1 is the promise return_xxxx call for for the expression given. */
Op1 is the promise return_xxxx call for the expression given. */

DEFTREECODE (CO_RETURN_EXPR, "co_return", tcc_statement, 2)

Expand Down
2 changes: 1 addition & 1 deletion gcc/cp/parser.cc
Original file line number Diff line number Diff line change
Expand Up @@ -45273,7 +45273,7 @@ substitute_in_tree (tree *context, tree orig, tree repl, bool flatten)
}

/* Walker to patch up the BLOCK_NODE hierarchy after the above surgery.
*DP is is the parent block. */
*DP is the parent block. */

static tree
fixup_blocks_walker (tree *tp, int *walk_subtrees, void *dp)
Expand Down
4 changes: 2 additions & 2 deletions gcc/cp/pt.cc
Original file line number Diff line number Diff line change
Expand Up @@ -25503,7 +25503,7 @@ more_specialized_fn (tree pat1, tree pat2, int len)
if (DECL_STATIC_FUNCTION_P (decl1) || DECL_STATIC_FUNCTION_P (decl2))
{
/* Note C++20 DR2445 extended the above to static member functions, but
I think think the old G++ behavior of just skipping the object
I think the old G++ behavior of just skipping the object
parameter when comparing to a static member function was better, so
let's stick with that for now. This is CWG2834. --jason 2023-12 */
if (DECL_OBJECT_MEMBER_FUNCTION_P (decl1))
Expand Down Expand Up @@ -30564,7 +30564,7 @@ type_targs_deducible_from (tree tmpl, tree type)

/* Maybe add in default template args. This seems like a flaw in the
specification in terms of partial specialization, since it says the
partial specialization has the the template parameter list of A, but a
partial specialization has the template parameter list of A, but a
partial specialization can't have default targs. */
targs = coerce_template_parms (tparms, targs, tmpl, tf_none);
if (targs == error_mark_node)
Expand Down
4 changes: 2 additions & 2 deletions gcc/cp/semantics.cc
Original file line number Diff line number Diff line change
Expand Up @@ -3895,7 +3895,7 @@ finish_template_decl (tree parms)
// typename pair<T, U>::first_type void f(T, U);
//
// Here, it is unlikely that there is a partial specialization of
// pair constrained for for Integral and Floating_point arguments.
// pair constrained for Integral and Floating_point arguments.
//
// The general rule is: if a constrained specialization with matching
// constraints is found return that type. Also note that if TYPE is not a
Expand Down Expand Up @@ -10692,7 +10692,7 @@ finish_omp_for (location_t locus, enum tree_code code, tree declv,
initializer is a binding of the iteration variable, save
that location. Any of these locations in the initialization clause
for the current nested loop are better than using the argument locus,
that points to the "for" of the the outermost loop in the nest. */
that points to the "for" of the outermost loop in the nest. */
if (init && EXPR_HAS_LOCATION (init))
elocus = EXPR_LOCATION (init);
else if (decl && INDIRECT_REF_P (decl) && EXPR_HAS_LOCATION (decl))
Expand Down
4 changes: 2 additions & 2 deletions gcc/doc/invoke.texi
Original file line number Diff line number Diff line change
Expand Up @@ -11710,7 +11710,7 @@ By default the analyzer attempts to reconstruct the original function
frames, and to emit events showing the inlined calls.

With @option{-fno-analyzer-undo-inlining} this attempt to reconstruct
the original frame information can be be disabled, which may be of help
the original frame information can be disabled, which may be of help
when debugging issues in the analyzer.

@item -fanalyzer-verbose-edges
Expand Down Expand Up @@ -25759,7 +25759,7 @@ Outputs pseudo-c assembly dialect.
@item -minline-memops-threshold=@var{bytes}
Specifies a size threshold in bytes at or below which memmove, memcpy
and memset shall always be expanded inline. Operations dealing with
sizes larger than this threshold would have to be be implemented using
sizes larger than this threshold would have to be implemented using
a library call instead of being expanded inline, but since BPF doesn't
allow libcalls, exceeding this threshold results in a compile-time
error. The default is @samp{1024} bytes.
Expand Down
6 changes: 3 additions & 3 deletions gcc/doc/options.texi
Original file line number Diff line number Diff line change
Expand Up @@ -422,9 +422,9 @@ The option is the inverse of another option that has the
the options-processing script will declare @code{TARGET_@var{thisname}},
@code{TARGET_@var{name}_P} and @code{TARGET_@var{name}_OPTS_P} macros:
@code{TARGET_@var{thisname}} is 1 when the option is active and 0 otherwise,
@code{TARGET_@var{name}_P} is similar to @code{TARGET_@var{name}} but take an
argument as @samp{target_flags}, and and @code{TARGET_@var{name}_OPTS_P} also
similar to @code{TARGET_@var{name}} but take an argument as @code{gcc_options}.
@code{TARGET_@var{name}_P} is similar to @code{TARGET_@var{name}} but takes an
argument as @samp{target_flags}, and @code{TARGET_@var{name}_OPTS_P} is also
similar to @code{TARGET_@var{name}} but takes an argument as @code{gcc_options}.

@item Enum(@var{name})
The option's argument is a string from the set of strings associated
Expand Down
2 changes: 1 addition & 1 deletion gcc/expr.cc
Original file line number Diff line number Diff line change
Expand Up @@ -392,7 +392,7 @@ convert_mode_scalar (rtx to, rtx from, int unsignedp)
BFmode -> SFmode -> HFmode conversion where SFmode
has superset of BFmode values. We don't need
to handle sNaNs by raising exception and turning
into into qNaN though, as that can be done in the
it into qNaN though, as that can be done in the
SFmode -> HFmode conversion too. */
rtx temp = gen_reg_rtx (SFmode);
int save_flag_finite_math_only = flag_finite_math_only;
Expand Down
2 changes: 1 addition & 1 deletion gcc/function.h
Original file line number Diff line number Diff line change
Expand Up @@ -271,7 +271,7 @@ struct GTY(()) function {
htab_t GTY((skip)) value_histograms;

/* Annotated gconds so that basic conditions in the same expression map to
the same same uid. This is used for condition coverage. */
the same uid. This is used for condition coverage. */
hash_map <gcond*, unsigned> *GTY((skip)) cond_uids;

/* For function.cc. */
Expand Down
2 changes: 1 addition & 1 deletion gcc/gcov.cc
Original file line number Diff line number Diff line change
Expand Up @@ -2567,7 +2567,7 @@ add_branch_counts (coverage_info *coverage, const arc_info *arc)
}
}

/* Increment totals in COVERAGE according to to block BLOCK. */
/* Increment totals in COVERAGE according to block BLOCK. */

static void
add_condition_counts (coverage_info *coverage, const block_info *block)
Expand Down
2 changes: 1 addition & 1 deletion gcc/gimple-range-phi.cc
Original file line number Diff line number Diff line change
Expand Up @@ -325,7 +325,7 @@ phi_analyzer::operator[] (tree name)
return m_tab[v];
}

// Process phi node PHI to see if it it part of a group.
// Process phi node PHI to see if it is part of a group.

void
phi_analyzer::process_phi (gphi *phi)
Expand Down
2 changes: 1 addition & 1 deletion gcc/jit/docs/topics/expressions.rst
Original file line number Diff line number Diff line change
Expand Up @@ -238,7 +238,7 @@ Constructor expressions
The fields in ``fields`` need to be the same objects that were used
to create the struct.

Each value has to have have the same unqualified type as the field
Each value has to have the same unqualified type as the field
it is applied to.

A NULL value element in ``values`` is a shorthand for zero initialization
Expand Down
2 changes: 1 addition & 1 deletion gcc/mode-switching.cc
Original file line number Diff line number Diff line change
Expand Up @@ -585,7 +585,7 @@ forward_transfer (int bb_index)
return true;
}

/* A backwards confluence function. Update the the bb_info single_succ
/* A backwards confluence function. Update the bb_info single_succ
field for E's source block, based on changes to E's destination block.
At the end of the dataflow problem, single_succ is the single mode
that all successors require (directly or indirectly), or no_mode
Expand Down
2 changes: 1 addition & 1 deletion gcc/rtl-ssa/accesses.h
Original file line number Diff line number Diff line change
Expand Up @@ -358,7 +358,7 @@ class use_info : public access_info
use_info *next_any_insn_use () const;

// Return the next use by a debug instruction, or null if none.
// This is only valid if if is_in_debug_insn ().
// This is only valid if is_in_debug_insn ().
use_info *next_debug_insn_use () const;

// Return the previous use by a phi node in the list, or null if none.
Expand Down
4 changes: 2 additions & 2 deletions gcc/tree-profile.cc
Original file line number Diff line number Diff line change
Expand Up @@ -390,7 +390,7 @@ condition_uid (struct function *fn, basic_block b)
|/ \
T F

T has has multiple incoming edges and is the outcome of a short circuit,
T has multiple incoming edges and is the outcome of a short circuit,
with top = a, bot = b. The top node (a) is masked when the edge (b, T) is
taken.

Expand Down Expand Up @@ -437,7 +437,7 @@ condition_uid (struct function *fn, basic_block b)
The masking table is represented as two bitfields per term in the expression
with the index corresponding to the term in the Boolean expression.
a || b && c becomes the term vector [a b c] and the masking table [a[0]
a[1] b[0] ...]. The kth bit of a masking vector is set if the the kth term
a[1] b[0] ...]. The kth bit of a masking vector is set if the kth term
is masked by taking the edge.

The out masks are in uint64_t (the practical maximum for gcov_type_node for
Expand Down
2 changes: 1 addition & 1 deletion gcc/tree-ssa-phiopt.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1251,7 +1251,7 @@ value_replacement (basic_block cond_bb, basic_block middle_bb,
_3 = i_6 != 0;
Here, carg is 4, oarg is 6, crhs is 0, and because
(4 != 0) == (6 != 0), we don't care if i_6 is 4 or 6, both
have the same outcome. So, can can optimize this to:
have the same outcome. So, we can optimize this to:
_3 = i_2(D) != 0;
If the single imm use of phi result >, >=, < or <=, similarly
we can check if both carg and oarg compare the same against
Expand Down
2 changes: 1 addition & 1 deletion gcc/tree-ssa-sccvn.cc
Original file line number Diff line number Diff line change
Expand Up @@ -5979,7 +5979,7 @@ visit_phi (gimple *phi, bool *inserted, bool backedges_varying_p)
if (SSA_NAME_OCCURS_IN_ABNORMAL_PHI (PHI_RESULT (phi)))
return set_ssa_val_to (PHI_RESULT (phi), PHI_RESULT (phi));

/* We track whether a PHI was CSEd to to avoid excessive iterations
/* We track whether a PHI was CSEd to avoid excessive iterations
that would be necessary only because the PHI changed arguments
but not value. */
if (!inserted)
Expand Down
2 changes: 1 addition & 1 deletion gcc/tree-vrp.cc
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@ remove_unreachable::maybe_register (gimple *s)
m_list.safe_push (std::make_pair (e->src->index, e->dest->index));
}

// Return true if all uses of NAME are dominated by by block BB. 1 use
// Return true if all uses of NAME are dominated by block BB. 1 use
// is allowed in block BB, This is one we hope to remove.
// ie
// _2 = _1 & 7;
Expand Down
2 changes: 1 addition & 1 deletion gcc/value-range.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1886,7 +1886,7 @@ irange_bitmask::adjust_range (irange &r) const
}
}

// If the the mask can be trivially converted to a range, do so and
// If the mask can be trivially converted to a range, do so and
// return TRUE.

bool
Expand Down
2 changes: 1 addition & 1 deletion gcc/vr-values.cc
Original file line number Diff line number Diff line change
Expand Up @@ -167,7 +167,7 @@ check_for_binary_op_overflow (range_query *query,
return true;
}

/* Set INIT, STEP, and DIRECTION the the corresponding values of NAME
/* Set INIT, STEP, and DIRECTION to the corresponding values of NAME
within LOOP, and return TRUE. Otherwise return FALSE, and set R to
the conservative range of NAME within the loop. */

Expand Down

0 comments on commit 64aa48c

Please sign in to comment.