diff options
| author | Thomas Schwinge <thomas@codesourcery.com> | 2021-08-11 22:31:55 +0200 |
|---|---|---|
| committer | Thomas Schwinge <thomas@codesourcery.com> | 2021-08-16 12:09:14 +0200 |
| commit | 7b9d99e615212c24cecae4202d8def9aa5e71809 (patch) | |
| tree | e02a394330940ebb4f3c22503960324b23b7dfde | |
| parent | 049eda8274b7394523238b17ab12c3e2889f253e (diff) | |
| download | gcc-7b9d99e615212c24cecae4202d8def9aa5e71809.zip gcc-7b9d99e615212c24cecae4202d8def9aa5e71809.tar.gz gcc-7b9d99e615212c24cecae4202d8def9aa5e71809.tar.bz2 | |
Clarify memory management for 'prop_set' in 'gcc/omp-oacc-neuter-broadcast.cc'
Clean-up for recent commit e2a58ed6dc5293602d0d168475109caa81ad0f0d
"openacc: Middle-end worker-partitioning support".
gcc/
* omp-oacc-neuter-broadcast.cc
(execute_omp_oacc_neuter_broadcast): Clarify memory management for
'prop_set'.
| -rw-r--r-- | gcc/omp-oacc-neuter-broadcast.cc | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/gcc/omp-oacc-neuter-broadcast.cc b/gcc/omp-oacc-neuter-broadcast.cc index 9bde0ac..d308670 100644 --- a/gcc/omp-oacc-neuter-broadcast.cc +++ b/gcc/omp-oacc-neuter-broadcast.cc @@ -1398,11 +1398,8 @@ execute_omp_oacc_neuter_broadcast () FOR_ALL_BB_FN (bb, cfun) bb->aux = NULL; - vec<propagation_set *> prop_set; - prop_set.create (last_basic_block_for_fn (cfun)); - - for (int i = 0; i < last_basic_block_for_fn (cfun); i++) - prop_set.quick_push (0); + vec<propagation_set *> prop_set (vNULL); + prop_set.safe_grow_cleared (last_basic_block_for_fn (cfun), true); find_ssa_names_to_propagate (par, mask, worker_single, vector_single, &prop_set); @@ -1461,6 +1458,9 @@ execute_omp_oacc_neuter_broadcast () delete it.second; record_field_map.empty (); + /* These are supposed to have been 'delete'd by 'neuter_worker_single'. */ + for (auto it : prop_set) + gcc_checking_assert (!it); prop_set.release (); /* This doesn't seem to make a difference. */ |