From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 101152 invoked by alias); 9 Jun 2015 11:07:18 -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 101142 invoked by uid 89); 9 Jun 2015 11:07:18 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.3 required=5.0 tests=AWL,BAYES_50,KAM_LAZY_DOMAIN_SECURITY,T_RP_MATCHES_RCVD autolearn=no version=3.3.2 X-HELO: mx2.suse.de Received: from cantor2.suse.de (HELO mx2.suse.de) (195.135.220.15) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (CAMELLIA256-SHA encrypted) ESMTPS; Tue, 09 Jun 2015 11:07:16 +0000 Received: from relay1.suse.de (charybdis-ext.suse.de [195.135.220.254]) by mx2.suse.de (Postfix) with ESMTP id 1862DACF7; Tue, 9 Jun 2015 11:07:13 +0000 (UTC) Date: Tue, 09 Jun 2015 11:12:00 -0000 From: Richard Biener To: Tom de Vries cc: GCC Patches Subject: Re: [PING^2][PATCH][3/3][PR65460] Mark offloaded functions as parallelized In-Reply-To: <557588BA.6060409@mentor.com> Message-ID: References: <5509B43E.802@mentor.com> <550AAD8B.9080900@mentor.com> <550C06BB.1050500@mentor.com> <5530DBA0.2000703@mentor.com> <557588BA.6060409@mentor.com> User-Agent: Alpine 2.11 (LSU 23 2013-08-11) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII X-SW-Source: 2015-06/txt/msg00649.txt.bz2 On Mon, 8 Jun 2015, Tom de Vries wrote: > On 17/04/15 12:08, Tom de Vries wrote: > > On 20-03-15 12:38, Tom de Vries wrote: > > > 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? > > > > > > > ping. > > ping^2. Original post at > https://gcc.gnu.org/ml/gcc-patches/2015-03/msg01063.html . Ok, but shouldn't it be set before calling add_new_function as add_new_function might run passes that wouldn't identify the function as parallelized? Richard. > > > > OK for stage1? > > > > Thanks, > - Tom > > > -- Richard Biener SUSE LINUX GmbH, GF: Felix Imendoerffer, Jane Smithard, Dilip Upmanyu, Graham Norton, HRB 21284 (AG Nuernberg)