This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH], Add power9 support to GCC, patch #3 (scalar count trailing zeros)
- From: Segher Boessenkool <segher at kernel dot crashing dot org>
- To: Michael Meissner <meissner at linux dot vnet dot ibm dot com>, gcc-patches at gcc dot gnu dot org, dje dot gcc at gmail dot com
- Date: Mon, 9 Nov 2015 13:33:11 -0600
- Subject: Re: [PATCH], Add power9 support to GCC, patch #3 (scalar count trailing zeros)
- Authentication-results: sourceware.org; auth=none
- References: <20151103202911 dot GA5304 at ibm-tiger dot the-meissners dot org> <20151109003753 dot GC17170 at ibm-tiger dot the-meissners dot org> <20151109155942 dot GA8759 at gate dot crashing dot org> <20151109171748 dot GA32705 at ibm-tiger dot the-meissners dot org>
On Mon, Nov 09, 2015 at 12:17:49PM -0500, Michael Meissner wrote:
> > > + "TARGET_CTZ"
> > > + "cnttz<wd> %0,%1"
> > > + [(set_attr "type" "cntlz")])
> >
> > We should probably rename this attr value now. "cntz" maybe? Could be
> > later of course.
>
> I don't see a need to add another type attribute for count trailing zeros
> unless count leading zeros has a different timing than count trailing zeros.
I didn't suggest adding a "cnttz"; I suggested renaming "cntlz". Maybe
"ctz" is better, that's what the target flag is as well.
Cheers,
Segher