Re: [PATCH 5/6] mips: Implement vec_perm_const.

[Hans-Peter Nilsson <hp at bitrange dot com>]

On Thu, 8 Dec 2011, Richard Henderson wrote:
> diff --git a/gcc/config/mips/mips.c b/gcc/config/mips/mips.c
> index d3fd709..f1c3665 100644
> --- a/gcc/config/mips/mips.c
> +++ b/gcc/config/mips/mips.c

> @@ -13021,8 +13015,8 @@ static const struct mips_builtin_description mips_builtins[] = {
>    LOONGSON_BUILTIN (pasubub, MIPS_UV8QI_FTYPE_UV8QI_UV8QI),
>    LOONGSON_BUILTIN (biadd, MIPS_UV4HI_FTYPE_UV8QI),
>    LOONGSON_BUILTIN (psadbh, MIPS_UV4HI_FTYPE_UV8QI_UV8QI),
> -  LOONGSON_BUILTIN_SUFFIX (pshufh, u, MIPS_UV4HI_FTYPE_UV4HI_UV4HI_UQI),
> -  LOONGSON_BUILTIN_SUFFIX (pshufh, s, MIPS_V4HI_FTYPE_V4HI_V4HI_UQI),
> +  LOONGSON_BUILTIN_SUFFIX (pshufh, u, MIPS_UV4HI_FTYPE_UV4HI_UQI),
> +  LOONGSON_BUILTIN_SUFFIX (pshufh, s, MIPS_V4HI_FTYPE_V4HI_UQI),
>    LOONGSON_BUILTIN_SUFFIX (psllh, u, MIPS_UV4HI_FTYPE_UV4HI_UQI),

Looks like a brute-force (ignoring backward compatibility) fix
for PR48068 item 2.  If going that route, I'd suggest at least
increment the __mips_loongson_vector_rev.  Also, loongson.h
needs the corresponding adjustment.

(No specific interest in Loongson, FWIW.)

brgds, H-P