From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 81542 invoked by alias); 19 Mar 2015 11:05:57 -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 81500 invoked by uid 89); 19 Mar 2015 11:05:56 -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; Thu, 19 Mar 2015 11:05:55 +0000 Received: from nat-ies.mentorg.com ([192.94.31.2] helo=SVR-IES-FEM-01.mgc.mentorg.com) by relay1.mentorg.com with esmtp id 1YYYGu-0004UZ-CH from Tom_deVries@mentor.com ; Thu, 19 Mar 2015 04:05:52 -0700 Received: from [127.0.0.1] (137.202.0.76) by SVR-IES-FEM-01.mgc.mentorg.com (137.202.0.104) with Microsoft SMTP Server id 14.3.224.2; Thu, 19 Mar 2015 11:05:50 +0000 Message-ID: <550AAD8B.9080900@mentor.com> Date: Thu, 19 Mar 2015 11:05: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> In-Reply-To: <5509B43E.802@mentor.com> Content-Type: multipart/mixed; boundary="------------060109010600050602020007" X-SW-Source: 2015-03/txt/msg01002.txt.bz2 --------------060109010600050602020007 Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit Content-length: 471 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. Bootstrapped and reg-tested on x86_64. OK for stage4 trunk? Thanks, - Tom --------------060109010600050602020007 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: 777 Mark offloaded functions as parallelized 2015-03-18 Tom de Vries PR tree-optimization/65460 * omp-low.c (expand_omp_target): Call mark_parallelized_function 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 a49a6eb..7195aa3 100644 --- a/gcc/omp-low.c +++ b/gcc/omp-low.c @@ -8938,6 +8938,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); + mark_parallelized_function (child_fn); #ifdef ENABLE_OFFLOADING /* Add the new function to the offload table. */ -- 1.9.1 --------------060109010600050602020007--