On Wed, Jun 08, 2016 at 03:34:53PM -0600, Kelvin Nilsen wrote:
> This patch adds built-in function support for the ISA 3.0 vabsub,
> vabsduh, and vabsduw instructions. This second version of the patch
> differs from the first in the following ways:
> I have bootstrapped and tested on powerpc64le-unknown-linux-gnu with no
> regressions. Is this ok for the trunk?
>
> I have also tested against the gcc-6 branch without regressions. Is
> this ok for backporting to gcc6 after a few days of burn-in time on the
> trunk?
> 2016-06-08 Kelvin Nilsen <kel...@gcc.gnu.org>
>
> * config/rs6000/altivec.h (vec_absd): New macro for vector absolute
> difference unsigned.
> (vec_absdb): New macro for vector absolute difference unsigned
> byte.
> (vec_absdh): New macro for vector absolute difference unsigned
> half-word.
Trailing space.
> +;; Vector absolute difference unsigned
> +(define_expand "vadu<mode>3"
> + [(set (match_operand:VI 0 "register_operand" "")
Missed one "" here.
> +The following built-in functions are available for the PowerPC family
> +of processors, starting with ISA 3.0 or later (@option{-mcpu=power9})
> +or with @option{-mpower9-vector}:
> +@smallexample
> +__vector unsigned char
> +vec_absd (__vector unsigned char arg1, __vector unsigned char arg2);
> +__vector unsighed short
Typo.
Okay for trunk with those things fixed; okay for 6 some time later.
Thanks,
Segher
