This is the mail archive of the fortran@gcc.gnu.org mailing list for the GNU Fortran project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [gfortran,patch] Don't call library function for string copying, take 2

From: "Richard Guenther" <richard dot guenther at gmail dot com>
To: "François-Xavier Coudert" <fxcoudert at gmail dot com>
Cc: gfortran <fortran at gcc dot gnu dot org>, patch <gcc-patches at gcc dot gnu dot org>
Date: Tue, 4 Jul 2006 16:47:07 +0200
Subject: Re: [gfortran,patch] Don't call library function for string copying, take 2
Domainkey-signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=KVGO3eERQTTILxQhxhkJx14O6GohCpj/OYZfgMYjkdf3a5dKKtzsmGp7R0XwBrV4qR7o0TNuOU+1YjXBJKtrHfoJy9GUoYdAelpZZtnXVB3swk9yOXL5F08Nx9KBTvo8/57o823DX16GixHTSpWkJC7vGV9TTb3fEoSaqh0lD/Q=
References: <19c433eb0607040744o491fa334t91f3483a58e7118@mail.gmail.com>

On 7/4/06, François-Xavier Coudert <fxcoudert@gmail.com> wrote:

:ADDPATCH fortran:

Attached patch is the second take on PR fortran/28163. Instead of
generating calls to libgfortran copy_string, we emit the code
directly. As my previous mails showed, the size impact is minimal
since the code emitted is small. There is a huge (positive)
performance impact of this on at least one of the polyhedron
benchmarks, aermod.

The differences with my first version of the patch are:
  -- I propose to remove the copy_string function from libgfortran,
since it's not used anywhere else


You need to bump the .so version of libgfortran if you do this (maybe we
have done so for 4.2 compared to 4.1.x already).

Richard.

Follow-Ups:
- Re: [gfortran,patch] Don't call library function for string copying, take 2
  - From: François-Xavier Coudert

References:
- [gfortran,patch] Don't call library function for string copying, take 2
  - From: François-Xavier Coudert

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]