Hi! 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! Segher
