From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 125609 invoked by alias); 20 Mar 2015 11:38:44 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 125599 invoked by uid 89); 20 Mar 2015 11:38:44 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.3 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 X-HELO: relay1.mentorg.com Received: from relay1.mentorg.com (HELO relay1.mentorg.com) (192.94.38.131) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 20 Mar 2015 11:38:43 +0000 Received: from nat-ies.mentorg.com ([192.94.31.2] helo=SVR-IES-FEM-02.mgc.mentorg.com) by relay1.mentorg.com with esmtp id 1YYvGC-0002F9-2u from Tom_deVries@mentor.com ; Fri, 20 Mar 2015 04:38:40 -0700 Received: from [127.0.0.1] (137.202.0.76) by SVR-IES-FEM-02.mgc.mentorg.com (137.202.0.106) with Microsoft SMTP Server id 14.3.224.2; Fri, 20 Mar 2015 11:38:38 +0000 Message-ID: <550C06BB.1050500@mentor.com> Date: Fri, 20 Mar 2015 11:38:00 -0000 From: Tom de Vries User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 MIME-Version: 1.0 To: GCC Patches , Richard Biener Subject: Re: [PATCH][3/3][PR65460] Mark offloaded functions as parallelized References: <5509B43E.802@mentor.com> <550AAD8B.9080900@mentor.com> In-Reply-To: <550AAD8B.9080900@mentor.com> Content-Type: multipart/mixed; boundary="------------050509020902030308000803" X-SW-Source: 2015-03/txt/msg01063.txt.bz2 --------------050509020902030308000803 Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit Content-length: 568 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 --------------050509020902030308000803 Content-Type: text/x-patch; name="0003-Mark-offloaded-functions-as-parallelized.patch" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="0003-Mark-offloaded-functions-as-parallelized.patch" Content-length: 803 Mark offloaded functions as parallelized 2015-03-20 Tom de Vries 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 --------------050509020902030308000803--