On 9/7/26 8:43 AM, [email protected] wrote:
From: Kyrylo Tkachov <[email protected]> A round of comments that do not match the code they document. noce_try_cmove names noce_try_store_flag_arith, a function that has never existed. block_jumps_and_fallthru says it returns true, but it returns an insn count or -1. dead_or_predicable documents every parameter but OTHER_BB, noce_convert_multiple_sets_1 does not document the condition this series started passing in, and noce_emit_cmove is described only as a helper of two of its five callers. get_base_reg_or_constant describes only the register case, and init_noce_multiple_sets_info never says that it fills INSN_INFO. The noce_try_cond_arith comment is reflowed to the file's style, and the rest are typos, a duplicated "(non-jump)", and a stray trailing space. Comment-only. gcc/ChangeLog: * ifcvt.cc (first_active_insn, find_active_insn_after, block_fallthru, noce_try_move, noce_try_sign_bit_splat, noce_try_store_flag_mask, noce_emit_cmove, noce_try_cmove, get_base_reg_or_constant, noce_try_cond_arith, try_emit_cmove_seq, noce_convert_multiple_sets_1, init_noce_multiple_sets_info, check_cond_move_block, find_if_header, block_jumps_and_fallthru, find_if_case_2, dead_or_predicable): Fix comments. Signed-off-by: Kyrylo Tkachov <[email protected]>
OK jeff
