This patch removes a dead if statement checking for gomp-parallel gimple
statements. This if is in the execute method of build_cgraph_edges pass,
which is executed right after the omp_expand pass, which removes these
gimple statements and replaces them with simple gcalls, making this if
practically dead.
Some TSan tests are failing with this patch, but I don't think this
change is likely to cause these failures. Additionally, the failures are
not consistent across runs, making me think these failures are a
bug in TSan itself. All other tests are ok. Tested on x86_64-pc-linux-gnu.
OK for master ?
gcc/ChangeLog:
* cgraphbuild.cc (pass_build_cgraph_edges::execute): remove if
statement checking for gomp-parallel statements
Signed-off-by: Josef Melcr <melcr...@fit.cvut.cz>
---
gcc/cgraphbuild.cc | 6 ------
1 file changed, 6 deletions(-)
diff --git a/gcc/cgraphbuild.cc b/gcc/cgraphbuild.cc
index 8852ba9ee92..191ec1077e4 100644
--- a/gcc/cgraphbuild.cc
+++ b/gcc/cgraphbuild.cc
@@ -340,12 +340,6 @@ pass_build_cgraph_edges::execute (function *fun)
bb->count);
}
node->record_stmt_references (stmt);
- if (gomp_parallel *omp_par_stmt = dyn_cast <gomp_parallel *> (stmt))
- {
- tree fn = gimple_omp_parallel_child_fn (omp_par_stmt);
- node->create_reference (cgraph_node::get_create (fn),
- IPA_REF_ADDR, stmt);
- }
if (gimple_code (stmt) == GIMPLE_OMP_TASK)
{
tree fn = gimple_omp_task_child_fn (stmt);
--
2.47.0
