Hi,

this patch removes a workaround that's no longer needed, now that we split off the kernels region at the first omp-expand pass.

Committed to gomp-4_0-branch.

Thanks,
- Tom
>From 5e9a609006b45c51598a3d52d5ab55b72a186f67 Mon Sep 17 00:00:00 2001
From: Tom de Vries <t...@codesourcery.com>
Date: Fri, 6 Nov 2015 22:10:31 +0100
Subject: [PATCH 1/4] Undo cgraph_node::release_body workaround

2015-11-07  Tom de Vries  <t...@codesourcery.com>

	* cgraph.c (cgraph_node::release_body): Remove workaround.
---
 gcc/cgraph.c | 9 ---------
 1 file changed, 9 deletions(-)

diff --git a/gcc/cgraph.c b/gcc/cgraph.c
index 8fe1ab4..7839c72 100644
--- a/gcc/cgraph.c
+++ b/gcc/cgraph.c
@@ -1707,15 +1707,6 @@ release_function_body (tree decl)
 void
 cgraph_node::release_body (bool keep_arguments)
 {
-  /* The omp-expansion of the oacc kernels directive is post-poned till after
-     all_small_ipa_passes.  That means pass_ipa_free_lang_data, which tries to
-     release the body of the offload function, is run before omp_expand_target 
-     can process the oacc kernels directive,  and omp_expand_target would crash
-     trying to access the body.  This snippet works around this problem.
-     FIXME: This should probably be fixed in a different way.  */
-  if (offloadable)
-    return;
-
   ipa_transforms_to_apply.release ();
   if (!used_as_abstract_origin && symtab->state != PARSING)
     {
-- 
1.9.1

Reply via email to