This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC 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: [PATCH, middle-end]: Convert (int)logb() into ilogb().

From: "Richard Guenther" <richard dot guenther at gmail dot com>
To: "Uros Bizjak" <ubizjak at gmail dot com>
Cc: "Paolo Bonzini" <paolo dot bonzini at lu dot unisi dot ch>, "GCC Patches" <gcc-patches at gcc dot gnu dot org>
Date: Wed, 31 Jan 2007 12:56:19 +0100
Subject: Re: [PATCH, middle-end]: Convert (int)logb() into ilogb().
References: <5787cf470701310028r4081b607naa7bd7f6a48f2950@mail.gmail.com> <45C05AAF.5060909@lu.unisi.ch> <5787cf470701310158o784f69cco68e92c7ddbb89b16@mail.gmail.com>

On 1/31/07, Uros Bizjak <ubizjak@gmail.com> wrote:

On 1/31/07, Paolo Bonzini <paolo.bonzini@lu.unisi.ch> wrote:

> > This patch implements a small optimization opportunity by converting
> > (int)logb() into ilogb() for -ffast-math.
>
> Cute.  Why not do so in fold-const.c?

Because there are no constants involved. ;)

Also, it converts floating point result into integer, so IMO its place
should be near (int)rint() -> lrint() code  in convert.c that does
exactly the same transformation.


That should also move to fold-const.c ... (in fact, all the BUILT_IN
tansformations
should).

Richard.

References:
- [PATCH, middle-end]: Convert (int)logb() into ilogb().
  - From: Uros Bizjak
- Re: [PATCH, middle-end]: Convert (int)logb() into ilogb().
  - From: Paolo Bonzini
- Re: [PATCH, middle-end]: Convert (int)logb() into ilogb().
  - From: Uros Bizjak

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