On Mon, Nov 09, 2015 at 12:17:49PM -0500, Michael Meissner wrote:
> > > + "TARGET_CTZ"
> > > + "cnttz %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 attr
On Sun, Nov 8, 2015 at 4:37 PM, Michael Meissner
wrote:
> This patch adds support for scalar count trailing zeros instruction that is
> being added to ISA 3.0 (power9).
>
> I have built this patch (along with patches #2 and #4) with a bootstrap build
> on a power8 little endian system. There were
On Mon, Nov 09, 2015 at 09:59:43AM -0600, Segher Boessenkool wrote:
> On Sun, Nov 08, 2015 at 07:37:53PM -0500, Michael Meissner wrote:
> > This patch adds support for scalar count trailing zeros instruction that is
> > being added to ISA 3.0 (power9).
>
> I bet you should change CTZ_DEFINED_VALUE
On Sun, Nov 08, 2015 at 07:37:53PM -0500, Michael Meissner wrote:
> This patch adds support for scalar count trailing zeros instruction that is
> being added to ISA 3.0 (power9).
I bet you should change CTZ_DEFINED_VALUE_AT_ZERO as well.
> +(define_insn "ctz2_hw"
> + [(set (match_operand:GPR 0 "
This patch adds support for scalar count trailing zeros instruction that is
being added to ISA 3.0 (power9).
I have built this patch (along with patches #2 and #4) with a bootstrap build
on a power8 little endian system. There were no regressions in the test
suite. Is this patch ok to install in
