From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 18538 invoked by alias); 13 Apr 2012 17:32:43 -0000 Received: (qmail 18448 invoked by uid 22791); 13 Apr 2012 17:32:41 -0000 X-SWARE-Spam-Status: No, hits=-4.9 required=5.0 tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,KHOP_RCVD_TRUST,KHOP_THREADED,RCVD_IN_DNSWL_LOW,RCVD_IN_HOSTKARMA_YE X-Spam-Check-By: sourceware.org Received: from mail-lb0-f175.google.com (HELO mail-lb0-f175.google.com) (209.85.217.175) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Fri, 13 Apr 2012 17:32:25 +0000 Received: by lbbgk1 with SMTP id gk1so53977lbb.20 for ; Fri, 13 Apr 2012 10:32:23 -0700 (PDT) MIME-Version: 1.0 Received: by 10.152.146.234 with SMTP id tf10mr2191218lab.31.1334338343425; Fri, 13 Apr 2012 10:32:23 -0700 (PDT) Received: by 10.152.14.36 with HTTP; Fri, 13 Apr 2012 10:32:23 -0700 (PDT) In-Reply-To: <4F8537F2.4050607@netcologne.de> References: <4F6DD691.9010907@netcologne.de> <4F8537F2.4050607@netcologne.de> Date: Fri, 13 Apr 2012 17:32:00 -0000 Message-ID: Subject: Re: [patch, fortran-dev] Use fixed variable sizes for stride calculations From: Janne Blomqvist To: Thomas Koenig Cc: "fortran@gcc.gnu.org" , gcc-patches Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable 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 X-SW-Source: 2012-04/txt/msg00857.txt.bz2 On Wed, Apr 11, 2012 at 10:51, Thomas Koenig wrote: > Hi, > > >> this patch uses division by known sizes (which can usually be replaced >> by a simple shift because intrinsics have sizes of power of two) instead >> of division by the size extracted from the array descriptor itself. >> >> This should save about 20 cycles for a single calculation. >> >> I'll go through the rest of the library to identify other possibilities >> for this. >> >> Regression-tested, no new failures. >> >> OK for the branch? > > > Full patch at http://gcc.gnu.org/ml/fortran/2012-03/msg00120.html > > Ping? > > =C2=A0 =C2=A0 =C2=A0 =C2=A0Thomas I thought I already approved this a few weeks ago (http://gcc.gnu.org/ml/fortran/2012-03/msg00144.html)? Or if it has been decided that I'm no longer a GFortran reviewer, it would be nice if I'd be informed of this as well. --=20 Janne Blomqvist