[PATCH] Fix for vcmpequt builtin

Segher Boessenkool segher@kernel.crashing.org
Mon Jun 14 21:35:26 GMT 2021


On Mon, Jun 14, 2021 at 01:50:57PM -0700, Carl Love wrote:
> The following patch removes some unused BU_P10_OVERLOAD_2 entries. 
> Also, it fixes the builtin definition to use the altivec_eqv1ti
> instruction definition for the compare instruction.

>  BU_P10V_AV_2 (VCMPGTUT,		"vcmpgtut",	CONST,	vector_gtuv1ti)
>  BU_P10V_AV_2 (VCMPGTST,		"vcmpgtst",	CONST,	vector_gtv1ti)
> -BU_P10V_AV_2 (VCMPEQUT,		"vcmpequt",	CONST,	eqvv1ti3)
> +BU_P10V_AV_2 (VCMPEQUT,		"vcmpequt",	CONST,	altivec_eqv1ti)

Could / should this use vector_eqv1ti instead?

Do you perhaps have a testcase for the eq/eqv mixup thing?  Don't spend
too much time on it, but maybe you have something.

Okay for trunk with that vector_eqv1ti change if that works (and is a
good idea etc.), and without it otherwise.  Thanks!


More information about the Gcc-patches mailing list