From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 13729 invoked by alias); 13 Nov 2015 13:25:43 -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 13661 invoked by uid 89); 13 Nov 2015 13:25:41 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.6 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 X-HELO: mail-wm0-f42.google.com Received: from mail-wm0-f42.google.com (HELO mail-wm0-f42.google.com) (74.125.82.42) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-GCM-SHA256 encrypted) ESMTPS; Fri, 13 Nov 2015 13:25:40 +0000 Received: by wmdw130 with SMTP id w130so28816540wmd.0 for ; Fri, 13 Nov 2015 05:25:37 -0800 (PST) MIME-Version: 1.0 X-Received: by 10.28.187.136 with SMTP id l130mr4039572wmf.100.1447421137340; Fri, 13 Nov 2015 05:25:37 -0800 (PST) Received: by 10.28.167.82 with HTTP; Fri, 13 Nov 2015 05:25:37 -0800 (PST) In-Reply-To: References: <908103EDB4893A42920B21D3568BFD9339F0AD26@MBX13.d.ethz.ch> Date: Fri, 13 Nov 2015 13:25:00 -0000 Message-ID: Subject: Re: [PATCH] RFC: Enable graphite at -O3 -fprofile_use From: aditya kumar To: Richard Biener Cc: VandeVondele Joost , "sebpop@gmail.com" , "gcc-patches@gcc.gnu.org" Content-Type: text/plain; charset=UTF-8 X-SW-Source: 2015-11/txt/msg01696.txt.bz2 Thanks all for supporting the idea of enabling graphite. We would collect compile-time, performance, code-size data and let you know. We are very positive that the compile time would have improved on an average, because of new scop detection algorithm. With the new set of patches graphite does not modify the code any more (if no optimizations have been done). This was one of the issues discussed at the Cauldron'15. Currently we have a couple more patches to put in before tomorrow, so collecting the data might take another few days, if that is okay. If we find serious regressions in any benchmark, we will address them as well. Aditya Kumar Compiler Engineer On Fri, Nov 13, 2015 at 5:32 AM, Richard Biener wrote: > On Fri, 13 Nov 2015, VandeVondele Joost wrote: > >> I'm all in favour of requiring isl and enabling graphite by default, but >> would suggest to enable it with -Ofast instead. >> >> One reason is that certainly extracting testcases from a PGO build is >> more difficult, and initially there will certainly be miscompiles with >> graphite (CP2K is right now). >> >> Furthermore, unless graphite is particularly effective with PGO (does it >> use average loop trip counts already?), I don't see a particular >> connection. > > The reason to choose FDO was so GRAPHITE can concentrate its computing > budget on the hot parts of a program (which profile estimation isn't > good enough identifying), reducing its compile-time cost. > > -Ofast isn't supposed to enable passes over -O3 so you're suggesting > to enable it with -O3 which I think is a bit premature. But we can > try doing that and revert at the end of stage3 if problems are just > too big. > > Richard.