Re: [PATCH][3/3][PR65460] Mark offloaded functions as parallelized

[Tom de Vries <Tom_deVries@mentor.com>]

[-- Attachment #1: Type: text/plain, Size: 568 bytes --]

On 19-03-15 12:05, Tom de Vries wrote:
> On 18-03-15 18:22, Tom de Vries wrote:
>> Hi,
>>
>> this patch fixes PR65460.
>>
>> The patch marks offloaded functions as parallelized, which means the parloops
>> pass no longer attempts to modify that function.
>
> Updated patch to postpone mark_parallelized_function until the corresponding
> cgraph_node is available, to ensure it works with the updated
> mark_parallelized_function from patch 2/3.
>

Updated to eliminate mark_parallelized_function.

Bootstrapped and reg-tested on x86_64.

OK for stage4?

Thanks,
- Tom

[-- Attachment #2: 0003-Mark-offloaded-functions-as-parallelized.patch --]
[-- Type: text/x-patch, Size: 803 bytes --]

Mark offloaded functions as parallelized

2015-03-20  Tom de Vries  <tom@codesourcery.com>

	PR tree-optimization/65460
	* omp-low.c (expand_omp_target): Set parallelized_function on
	cgraph_node for child_fn.
---
 gcc/omp-low.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/gcc/omp-low.c b/gcc/omp-low.c
index 5ca9e84..9be39b7 100644
--- a/gcc/omp-low.c
+++ b/gcc/omp-low.c
@@ -8937,6 +8937,7 @@ expand_omp_target (struct omp_region *region)
       /* Inform the callgraph about the new function.  */
       DECL_STRUCT_FUNCTION (child_fn)->curr_properties = cfun->curr_properties;
       cgraph_node::add_new_function (child_fn, true);
+      cgraph_node::get (child_fn)->parallelized_function = 1;
 
 #ifdef ENABLE_OFFLOADING
       /* Add the new function to the offload table.  */
-- 
1.9.1