diff options
| author | Richard Biener <rguenther@suse.de> | 2020-12-02 14:43:59 +0100 |
|---|---|---|
| committer | Richard Biener <rguenther@suse.de> | 2020-12-02 15:55:18 +0100 |
| commit | feb93adf76eda52385a73eb57c5bef7c870a2564 (patch) | |
| tree | ab341ac008fc8dbb5f504d4850d989a1f09bf142 /gcc/tree-vectorizer.c | |
| parent | bad800c03d00a57fc21718c160459d9a1e8d747a (diff) | |
| download | gcc-feb93adf76eda52385a73eb57c5bef7c870a2564.zip gcc-feb93adf76eda52385a73eb57c5bef7c870a2564.tar.gz gcc-feb93adf76eda52385a73eb57c5bef7c870a2564.tar.bz2 | |
tree-optimization/97630 - fix SLP cycle memory leak
This fixes SLP cycles leaking memory by maintaining a double-linked
list of allocatd SLP nodes we can zap when we free the alloc pool.
2020-12-02 Richard Biener <rguenther@suse.de>
PR tree-optimization/97630
* tree-vectorizer.h (_slp_tree::next_node,
_slp_tree::prev_node): New.
(vect_slp_init): Declare.
(vect_slp_fini): Likewise.
* tree-vectorizer.c (vectorize_loops): Call vect_slp_init/fini.
(pass_slp_vectorize::execute): Likewise.
* tree-vect-slp.c (vect_slp_init): New.
(vect_slp_fini): Likewise.
(slp_first_node): New global.
(_slp_tree::_slp_tree): Link node into the SLP tree list.
(_slp_tree::~_slp_tree): Delink node from the SLP tree list.
Diffstat (limited to 'gcc/tree-vectorizer.c')
| -rw-r--r-- | gcc/tree-vectorizer.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/gcc/tree-vectorizer.c b/gcc/tree-vectorizer.c index b63dda3..f9e2642 100644 --- a/gcc/tree-vectorizer.c +++ b/gcc/tree-vectorizer.c @@ -1171,7 +1171,7 @@ vectorize_loops (void) if (vect_loops_num <= 1) return 0; - slp_tree_pool = new object_allocator<_slp_tree> ("SLP nodes for vect"); + vect_slp_init (); if (cfun->has_simduid_loops) note_simd_array_uses (&simd_array_to_simduid_htab); @@ -1295,8 +1295,7 @@ vectorize_loops (void) shrink_simd_arrays (simd_array_to_simduid_htab, simduid_to_vf_htab); delete simduid_to_vf_htab; cfun->has_simduid_loops = false; - delete slp_tree_pool; - slp_tree_pool = NULL; + vect_slp_fini (); if (num_vectorized_loops > 0) { @@ -1432,12 +1431,11 @@ pass_slp_vectorize::execute (function *fun) } } - slp_tree_pool = new object_allocator<_slp_tree> ("SLP nodes for slp"); + vect_slp_init (); vect_slp_function (fun); - delete slp_tree_pool; - slp_tree_pool = NULL; + vect_slp_fini (); if (!in_loop_pipeline) { |