[PATCH 057/236] cfgcleanup.c: Use rtx_insn (also touches basic-block.h and ifcvt.c)

Jeff Law law@redhat.com
Wed Aug 13 18:23:00 GMT 2014


On 08/06/14 11:20, David Malcolm wrote:
> gcc/
> 	* basic-block.h (flow_find_cross_jump): Strengthen params 3 and 4
> 	"f1" and "f2" from rtx * to rtx_insn **.
> 	(flow_find_head_matching_sequence): Likewise.
>
> 	* cfgcleanup.c (try_simplify_condjump): Strengthen local
> 	"cbranch_insn" from rtx to rtx_insn *.
> 	(thread_jump): Likewise for local "insn".
> 	(try_forward_edges): Likewise for local "last".
> 	(merge_blocks_move_predecessor_nojumps): Likewise for local "barrier".
> 	(merge_blocks_move_successor_nojumps): Likewise for locals "barrier",
> 	"real_b_end".
> 	(can_replace_by): Likewise for params "i1", "i2".
> 	(old_insns_match_p): Likewise.
> 	(merge_notes): Likewise.
> 	(walk_to_nondebug_insn): Likewise for param "i1".
> 	(flow_find_cross_jump): Strengthen params "f1" and "f2" from rtx *
> 	to rtx_insn **.  Strengthen locals "i1", "i2", "last1", "last2",
> 	"afterlast1", "afterlast2" from rtx to rtx_insn *.
> 	(flow_find_head_matching_sequence): Strengthen params "f1" and
> 	"f2" from rtx * to rtx_insn **.  Strengthen locals "i1", "i2",
> 	"last1", "last2", "beforelast1", "beforelast2" from rtx to
> 	rtx_insn *.
> 	(outgoing_edges_match): Likewise for locals "last1", "last2".
> 	(try_crossjump_to_edge): Likewise for local "insn".
> 	Replace call to for_each_rtx with for_each_rtx_in_insn.
>
> 	(try_crossjump_to_edge): Likewise for locals "newpos1", "newpos2".
> 	(try_head_merge_bb): Likewise for locals "e0_last_head_, "jump",
> 	"e0_last", "e_last", "head", "curr", "insn".  Strengthen locals
> 	"headptr", "currptr", "nextptr" from rtx * to rtx_insn **.
> 	(try_optimize_cfg): Strengthen local "last" from rtx to
> 	rtx_insn *.
> 	(delete_dead_jumptables): Likewise for locals "insn", "next",
> 	"label".
>
> 	* ifcvt.c (cond_exec_process_if_block): Likewise for locals
> 	"rtx then_last_head", "rtx else_last_head", "rtx then_first_tail",
> 	"rtx else_first_tail", to reflect the basic-block.h changes above.
OK.  I noticed a for_each_rtx in here.  That block may need updating due 
to Richard S.'s changes.

Patches #58 & #59 are OK too.

jeff



More information about the Gcc-patches mailing list