diff options
| author | Richard Biener <rguenther@suse.de> | 2020-08-06 14:50:56 +0200 |
|---|---|---|
| committer | Richard Biener <rguenther@suse.de> | 2020-09-24 15:59:19 +0200 |
| commit | 4b9d61f79c0c0185a33048ae6cc72269cf7efa31 (patch) | |
| tree | f62956faf4f9fa1e7b25d44bee91258c462842fd /gcc/predict.c | |
| parent | 2e66e53b1efb98f5cf6b0a123990c1ca999affd7 (diff) | |
| download | gcc-4b9d61f79c0c0185a33048ae6cc72269cf7efa31.zip gcc-4b9d61f79c0c0185a33048ae6cc72269cf7efa31.tar.gz gcc-4b9d61f79c0c0185a33048ae6cc72269cf7efa31.tar.bz2 | |
add move CTOR to auto_vec, use auto_vec for get_loop_exit_edges
This adds a move CTOR to auto_vec<T, 0> and makes use of a
auto_vec<edge> return value for get_loop_exit_edges denoting
that lifetime management of the vector is handed to the caller.
The move CTOR prompted the hash_table change because it appearantly
makes the copy CTOR implicitely deleted (good) and hash-table
expansion of the odr_enum_map which is
hash_map <nofree_string_hash, odr_enum> where odr_enum has an
auto_vec<odr_enum_val, 0> member triggers this. Not sure if
there's a latent bug there before this (I think we're not
invoking DTORs, but we're invoking copy-CTORs).
2020-08-06 Richard Biener <rguenther@suse.de>
* vec.h (auto_vec<T, 0>::auto_vec (auto_vec &&)): New move CTOR.
(auto_vec<T, 0>::operator=(auto_vec &&)): Delete.
* hash-table.h (hash_table::expand): Use std::move when expanding.
* cfgloop.h (get_loop_exit_edges): Return auto_vec<edge>.
* cfgloop.c (get_loop_exit_edges): Adjust.
* cfgloopmanip.c (fix_loop_placement): Likewise.
* ipa-fnsummary.c (analyze_function_body): Likewise.
* ira-build.c (create_loop_tree_nodes): Likewise.
(create_loop_tree_node_allocnos): Likewise.
(loop_with_complex_edge_p): Likewise.
* ira-color.c (ira_loop_edge_freq): Likewise.
* loop-unroll.c (analyze_insns_in_loop): Likewise.
* predict.c (predict_loops): Likewise.
* tree-predcom.c (last_always_executed_block): Likewise.
* tree-ssa-loop-ch.c (ch_base::copy_headers): Likewise.
* tree-ssa-loop-im.c (store_motion_loop): Likewise.
* tree-ssa-loop-ivcanon.c (loop_edge_to_cancel): Likewise.
(canonicalize_loop_induction_variables): Likewise.
* tree-ssa-loop-manip.c (get_loops_exits): Likewise.
* tree-ssa-loop-niter.c (find_loop_niter): Likewise.
(finite_loop_p): Likewise.
(find_loop_niter_by_eval): Likewise.
(estimate_numbers_of_iterations): Likewise.
* tree-ssa-loop-prefetch.c (emit_mfence_after_loop): Likewise.
(may_use_storent_in_loop_p): Likewise.
Diffstat (limited to 'gcc/predict.c')
| -rw-r--r-- | gcc/predict.c | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/gcc/predict.c b/gcc/predict.c index 3c7b46f..5983889 100644 --- a/gcc/predict.c +++ b/gcc/predict.c @@ -1916,7 +1916,6 @@ predict_loops (void) { basic_block bb, *bbs; unsigned j, n_exits = 0; - vec<edge> exits; class tree_niter_desc niter_desc; edge ex; class nb_iter_bound *nb_iter; @@ -1927,15 +1926,12 @@ predict_loops (void) gcond *stmt = NULL; bool recursion = with_recursion.contains (loop); - exits = get_loop_exit_edges (loop); + auto_vec<edge> exits = get_loop_exit_edges (loop); FOR_EACH_VEC_ELT (exits, j, ex) if (!unlikely_executed_edge_p (ex) && !(ex->flags & EDGE_ABNORMAL_CALL)) n_exits ++; if (!n_exits) - { - exits.release (); - continue; - } + continue; if (dump_file && (dump_flags & TDF_DETAILS)) fprintf (dump_file, "Predicting loop %i%s with %i exits.\n", @@ -2049,7 +2045,6 @@ predict_loops (void) probability = RDIV (REG_BR_PROB_BASE, nitercst); predict_edge (ex, predictor, probability); } - exits.release (); /* Find information about loop bound variables. */ for (nb_iter = loop->bounds; nb_iter; |