This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PATCH: ffs/ctz expansions using clz
- From: Richard Sandiford <richard at codesourcery dot com>
- To: Sandra Loosemore <sandra at codesourcery dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 02 Aug 2007 07:25:48 +0100
- Subject: Re: PATCH: ffs/ctz expansions using clz
- References: <46B13576.2040800@codesourcery.com>
Sandra Loosemore <sandra@codesourcery.com> writes:
> This patch depends on CLZ_DEFINED_VALUE_AT_ZERO applying to the clz
> optab as well as the clz rtl expression. I couldn't tell from reading
> the documentation whether this is a change from its previous meaning
> or not, but I don't think it's inconsistent with current practice in
> all the other back ends.
It is a change. I was thinking the macro ought to be tristate:
defined always, defined for the clz rtl code only, or undefined.
Sorry for not making that clear.
(The patch looked good to me otherwise FWIW, although I obviously
can't approve it.)
Richard