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]

Re: [PATCH] Add ffs() to libiberty.

To: Richard Henderson <rth at redhat dot com>
Subject: Re: [PATCH] Add ffs() to libiberty.
From: Mark Klein <mklein at dis dot com>
Date: Mon, 09 Jul 2001 18:34:47 -0700
Cc: gcc-patches at gcc dot gnu dot org
References: <200107050559.f655xmc00908@dis.dis.com><200107050559.f655xmc00908@dis.dis.com>

At 02:39 PM 7/6/2001 -0700, Richard Henderson wrote:


>On Wed, Jul 04, 2001 at 10:59:48PM -0700, Mark Klein wrote:
> >   for (bit = 1; !(valu & 1); bit++)
> >       valu >>= 1;
>
>I'd rather see a binary search.  See bitmap.c (as of today)
>or longlong.h for examples.


Hi.

Is that a request to change the logic or simply a pointer to
an alternate algorithm for future reference?

--
Mark Klein                                 DIS International, Ltd.
http://www.dis.com                         415-892-8400
PGP Public Key Available

Follow-Ups:
- Re: [PATCH] Add ffs() to libiberty.
  - From: Richard Henderson

References:
- [PATCH] Add ffs() to libiberty.
  - From: Mark Klein
- Re: [PATCH] Add ffs() to libiberty.
  - From: Richard Henderson

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