On 9/7/26 8:43 AM, [email protected] wrote:
From: Kyrylo Tkachov <[email protected]> Of the 38 static forward declarations in ifcvt.cc, only ten name a function that is used before it is defined. The rest were needed by an earlier arrangement of the file: the whole mid-file block of noce_try_* declarations became dead when the ordered if-chain in noce_process_if_block turned into a table placed after every matcher. Delete the 28 dead ones and move noce_emit_cmove's default arguments onto its definition, since they lived on the declaration that goes away. No functional change. gcc/ChangeLog: * ifcvt.cc (count_bb_insns, cheap_bb_rtx_cost_p, first_active_insn, last_active_insn, find_active_insn_before, find_active_insn_after, block_fallthru, cond_exec_get_condition, noce_get_condition, find_if_header, block_jumps_and_fallthru, noce_find_if_block, noce_emit_move_insn, noce_emit_store_flag, noce_try_move, noce_try_ifelse_collapse, noce_try_store_flag, noce_try_addcc, noce_try_store_flag_constants, noce_try_shifted_store_flag, noce_try_store_flag_mask, noce_try_cmove, noce_try_cmove_arith, noce_get_alt_condition, noce_try_minmax, noce_try_abs, noce_try_sign_mask): Remove the unused forward declarations. (noce_emit_cmove): Likewise, and give the definition the default arguments. Signed-off-by: Kyrylo Tkachov <[email protected]>
OK jeff
