On Wed, Jun 06, 2018 at 03:30:17PM -0700, Carl Love wrote:
> > One thing that caught my eye, nothing new though:
> >
> > > {
> > > rtx tmp = gen_reg_rtx (SImode);
> > > - emit_insn (gen_vctzlsbb_<mode> (tmp, result));
> > > + if (!BYTES_BIG_ENDIAN)
> > > + emit_insn (gen_vctzlsbb_<mode> (tmp, result));
> > > + else
> > > + emit_insn (gen_vclzlsbb_<mode> (tmp, result));
> > > emit_insn (gen_ashrsi3 (operands[0], tmp, GEN_INT (sh)));
> > > }
> >
> > Can this work with lshrsi3 instead? That is slightly cheaper.
> >
> > Looks great, please apply to trunk. Thanks!
>
> Yes, the builtin works with lshrsi3 change. I will commit with this
> change.
It occurs in four places, please fix all then.
> FYI, Bill would like to patch backported to GCC 8. Is that OK?
Certainly. Thanks!
Segher
