On Wed, Jun 06, 2018 at 08:01:22AM -0700, Carl Love wrote:
> 2018-06-04 Carl Love <c...@us.ibm.com>
>
> * gcc/config/rs6000/vsx.md (first_match_index_<mode>):
> Calculate index using natureal element order.
Typo ("natural"). You normally shouldn't do a line break after : btw.
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!
Segher
