On 9/7/26 8:43 AM, [email protected] wrote:
From: Kyrylo Tkachov <[email protected]> Several field comments in ifcvt.h do not match the field they document. noce_multiple_sets_info::unmodified_insn holds the insn from the original block, not an insn we emitted, and need_cmov is documented with the opposite meaning: it is set when the destination is live out, that is, when a plain move will not do. noce_if_info::original_cost refers to a "next field" that no longer exists. Fix those, fix two typos, and make ce_if_block::and_and_p a bool, which is what it holds and how both ifcvt.cc and the frv target hooks read it. gcc/ChangeLog: * ifcvt.h (struct ce_if_block): Make and_and_p a bool. (struct noce_multiple_sets_info): Correct the unmodified_insn and need_cmov comments. (struct noce_if_info): Correct the insn_a/insn_b, speed_p and original_cost comments. Signed-off-by: Kyrylo Tkachov <[email protected]>
OK Jeff
