diff options
| author | Richard Biener <rguenther@suse.de> | 2022-08-02 13:46:28 +0200 |
|---|---|---|
| committer | Richard Biener <rguenther@suse.de> | 2022-08-09 10:26:55 +0200 |
| commit | 9aa08cd48490183382371bff4fd53573b5adabe8 (patch) | |
| tree | 03f3e0aca4d6084237878011caa483f215002909 /gcc/tree-parloops.cc | |
| parent | c64ef5cd92c2e340caabc95f4a28c9a125e8b5b8 (diff) | |
| download | gcc-9aa08cd48490183382371bff4fd53573b5adabe8.zip gcc-9aa08cd48490183382371bff4fd53573b5adabe8.tar.gz gcc-9aa08cd48490183382371bff4fd53573b5adabe8.tar.bz2 | |
autopar TLC
The following removes all excessive update_ssa calls from OMP
expansion, thereby rewriting the atomic load and store cases to
GIMPLE code generation. I don't think autopar ever exercises the
atomics code though.
There's not much test coverage overall so I've built SPEC 2k17
with -floop-parallelize-all -ftree-parallelize-loops=2 with and
without LTO (and otherwise -Ofast plus -march=haswell) without
fallout.
If there's any fallout it's not OK to update SSA form for
each and every OMP stmt lowered.
* omp-expand.cc (expand_omp_atomic_load): Emit GIMPLE
directly. Avoid update_ssa when in SSA form.
(expand_omp_atomic_store): Likewise.
(expand_omp_atomic_fetch_op): Avoid update_ssa when in SSA
form.
(expand_omp_atomic_pipeline): Likewise.
(expand_omp_atomic_mutex): Likewise.
* tree-parloops.cc (gen_parallel_loop): Use
TODO_update_ssa_no_phi after loop_version.
Diffstat (limited to 'gcc/tree-parloops.cc')
| -rw-r--r-- | gcc/tree-parloops.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gcc/tree-parloops.cc b/gcc/tree-parloops.cc index 2d3aa78..b070527 100644 --- a/gcc/tree-parloops.cc +++ b/gcc/tree-parloops.cc @@ -3082,7 +3082,7 @@ gen_parallel_loop (class loop *loop, profile_probability::unlikely (), profile_probability::likely (), profile_probability::unlikely (), true); - update_ssa (TODO_update_ssa); + update_ssa (TODO_update_ssa_no_phi); free_original_copy_tables (); } |