From 3d178f0c0c800997a69d8e2824c9cc8148a89549 Mon Sep 17 00:00:00 2001 From: Aldy Hernandez Date: Tue, 5 Sep 2017 20:48:25 +0000 Subject: [PATCH] tree-ssa-threadupdate.c (duplicate_thread_path): Remove unused REGION_COPY argument. * tree-ssa-threadupdate.c (duplicate_thread_path): Remove unused REGION_COPY argument. (thread_through_all_blocks): Remove unused argument to duplicate_thread_path. From-SVN: r251740 --- gcc/ChangeLog | 7 +++++++ gcc/tree-ssa-threadupdate.c | 21 +++++---------------- 2 files changed, 12 insertions(+), 16 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 3d38a7a..e5a8c5a 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,10 @@ +2017-09-05 Aldy Hernandez + + * tree-ssa-threadupdate.c (duplicate_thread_path): Remove unused + REGION_COPY argument. + (thread_through_all_blocks): Remove unused argument to + duplicate_thread_path. + 2017-09-05 Richard Sandiford Alan Hayward David Sherwood diff --git a/gcc/tree-ssa-threadupdate.c b/gcc/tree-ssa-threadupdate.c index 9da7467..d68223a 100644 --- a/gcc/tree-ssa-threadupdate.c +++ b/gcc/tree-ssa-threadupdate.c @@ -2205,18 +2205,13 @@ bb_in_bbs (basic_block bb, basic_block *bbs, int n) and create a single fallthru edge pointing to the same destination as the EXIT edge. - The new basic blocks are stored to REGION_COPY in the same order as they had - in REGION, provided that REGION_COPY is not NULL. - Returns false if it is unable to copy the region, true otherwise. */ static bool -duplicate_thread_path (edge entry, edge exit, - basic_block *region, unsigned n_region, - basic_block *region_copy) +duplicate_thread_path (edge entry, edge exit, basic_block *region, + unsigned n_region) { unsigned i; - bool free_region_copy = false; struct loop *loop = entry->dest->loop_father; edge exit_copy; edge redirected; @@ -2242,12 +2237,7 @@ duplicate_thread_path (edge entry, edge exit, set_loop_copy (loop, loop); - if (!region_copy) - { - region_copy = XNEWVEC (basic_block, n_region); - free_region_copy = true; - } - + basic_block *region_copy = XNEWVEC (basic_block, n_region); copy_bbs (region, n_region, region_copy, &exit, 1, &exit_copy, loop, split_edge_bb_loc (entry), false); @@ -2376,8 +2366,7 @@ duplicate_thread_path (edge entry, edge exit, /* Add the other PHI node arguments. */ add_phi_args_after_copy (region_copy, n_region, NULL); - if (free_region_copy) - free (region_copy); + free (region_copy); free_original_copy_tables (); return true; @@ -2502,7 +2491,7 @@ thread_through_all_blocks (bool may_peel_loop_headers) for (unsigned int j = 0; j < len - 1; j++) region[j] = (*path)[j]->e->dest; - if (duplicate_thread_path (entry, exit, region, len - 1, NULL)) + if (duplicate_thread_path (entry, exit, region, len - 1)) { /* We do not update dominance info. */ free_dominance_info (CDI_DOMINATORS); -- 2.7.4