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: "Uros Bizjak" <ubizjak at gmail dot com>
To: "Paolo Bonzini" <paolo dot bonzini at lu dot unisi dot ch>
Cc: "GCC Patches" <gcc-patches at gcc dot gnu dot org>
Date: Wed, 31 Jan 2007 10:58:02 +0100
Subject: Re: [PATCH, middle-end]: Convert (int)logb() into ilogb().
References: <5787cf470701310028r4081b607naa7bd7f6a48f2950@mail.gmail.com> <45C05AAF.5060909@lu.unisi.ch>

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.

Uros.

Follow-Ups:
- Re: [PATCH, middle-end]: Convert (int)logb() into ilogb().
  - From: Richard Guenther

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

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