From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 31150 invoked by alias); 20 Oct 2014 21:17:52 -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 31138 invoked by uid 89); 20 Oct 2014 21:17:51 -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,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 X-HELO: mail-wi0-f179.google.com Received: from mail-wi0-f179.google.com (HELO mail-wi0-f179.google.com) (209.85.212.179) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-SHA encrypted) ESMTPS; Mon, 20 Oct 2014 21:17:49 +0000 Received: by mail-wi0-f179.google.com with SMTP id d1so150503wiv.6 for ; Mon, 20 Oct 2014 14:17:46 -0700 (PDT) X-Received: by 10.194.236.232 with SMTP id ux8mr36967499wjc.96.1413839866733; Mon, 20 Oct 2014 14:17:46 -0700 (PDT) Received: from localhost ([95.144.14.223]) by mx.google.com with ESMTPSA id l5sm10912286wif.3.2014.10.20.14.17.45 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 20 Oct 2014 14:17:46 -0700 (PDT) From: Richard Sandiford To: Maxim Kuvyrkov Mail-Followup-To: Maxim Kuvyrkov ,Sebastian Pop , GCC Patches , Vladimir Makarov , Ramana Radhakrishnan , Marcus Shawcroft , Richard Earnshaw , rdsandiford@googlemail.com Cc: Sebastian Pop , GCC Patches , Vladimir Makarov , Ramana Radhakrishnan , Marcus Shawcroft , Richard Earnshaw Subject: Re: [PATCH] Account for prologue spills in reg_pressure scheduling References: <20141020191105.GB30024@f1.c.bardezibar.internal> <20141020204417.GA7213@f1.c.bardezibar.internal> <183D9AE9-ADFE-4E20-A8FF-B323A631FCC4@linaro.org> Date: Mon, 20 Oct 2014 21:21:00 -0000 In-Reply-To: <183D9AE9-ADFE-4E20-A8FF-B323A631FCC4@linaro.org> (Maxim Kuvyrkov's message of "Tue, 21 Oct 2014 09:56:34 +1300") Message-ID: <87wq7u30g6.fsf@googlemail.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-SW-Source: 2014-10/txt/msg01982.txt.bz2 Maxim Kuvyrkov writes: > [Adding ARM maintainers to CC] > > On Oct 21, 2014, at 9:44 AM, Sebastian Pop wrote: > >> Hi Maxim, >> >> Maxim Kuvyrkov wrote: >>> Thanks, benchmarking results are welcome! AArch64 doesn't use reg_pressure >>> scheduling by default. Use "-fsched-pressure >>> --param=sched-pressure-algorithm=2" to enable same thing as on ARM. I would >>> imagine C++ and Fortran floating-point code to be most affected. >> >> On aarch64 I only see perf improvements with your patch: no perf >> degradations on >> all the tests that I have run. >> >> base0: r216447, -O3 >> base1: r216447, -O3 -fsched-pressure --param=sched-pressure-algorithm=2 >> patch: r216447 + your patch, -O3 -fsched-pressure >> --param=sched-pressure-algorithm=2 >> >> patch vs. base1 is only an improvement. >> >> base1 vs. base0 has a few good improvements, and some small degradations: your >> patch improves the perf for one of the degradations to the point it is better >> now with -fsched-pressure --param=sched-pressure-algorithm=2 than at -O3. >> >> Could we turn on "-fsched-pressure --param=sched-pressure-algorithm=2" by >> default for aarch64? > > These are great results, yay! +1. Thanks for running these tests. If you have time, it'd also be interesting to try the same thing with --param=sched-pressure-algorithm=1 (which should be equivalent to not having the --param, but better safe than sorry). Is algorithm 1 or algorithm 2 better for aarch64? Thanks, Richard