This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: moving two speedups to 3.3 branch
- From: Segher Boessenkool <segher at koffie dot nl>
- To: Mike Stump <mstump at apple dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Sat, 08 Feb 2003 11:59:10 +0100
- Subject: Re: moving two speedups to 3.3 branch
- References: <B8799D14-3AF0-11D7-AC61-003065A77310@apple.com>
Mike Stump wrote:
2003-01-25 Segher Boessenkool <segher@koffie.nl>
* bitmap.h (BITMAP_WORD): New typedef: fundamental storage
type for bitmaps. Use unsigned long.
(nBITMAP_WORD_BITS): New macro.
(BITMAP_WORD_BITS): New macro.
(rest of file): Use it.
* bitmap.c: Use it.
I'd like these two for performance reasons. If Segher or Richard knows
of any reason why this would be bad, let us know.
Just be sure to not forget Richard's obvious fix for this patch:
Quoting him:
> Note that
>
> > - if (word & ~ (unsigned HOST_WIDE_INT) 0xffffffff)
> > + if (word & ~0xffffffff)
>
> is incorrect. The cast is still required for 64-bit hosts.
> I replaced it during the commit.
Thanks,
Segher