Re: [PATCH 15/33] target/ppc: Implement Vector Insert from GPR using GPR index insns

[Matheus K. Ferst]

On 23/10/2021 01:40, Richard Henderson wrote:
> [E-MAIL EXTERNO] Não clique em links ou abra anexos, a menos que você 
> possa confirmar o remetente e saber que o conteúdo é seguro. Em caso de 
> e-mail suspeito entre imediatamente em contato com o DTI.
>
> On 10/21/21 12:45 PM, matheus.ferst@eldorado.org.br wrote:
> > +#if defined(TARGET_PPC64)
> > +    return do_vinsx(ctx, a->vrt, size, right, cpu_gpr[a->vra], 
> > cpu_gpr[a->vrb],
> > +                    gen_helper);
> > +#else
> > +    bool ok;
> > +    TCGv_i64 val;
> > +
> > +    val = tcg_temp_new_i64();
> > +    tcg_gen_extu_tl_i64(val, cpu_gpr[a->vrb]);
> > +
> > +    ok = do_vinsx(ctx, a->vrt, size, right, cpu_gpr[a->vra], val, 
> > gen_helper);
> > +
> > +    tcg_temp_free_i64(val);
> > +    return ok;
> > +#endif
>
> Oh, and what's all this?
>
> Either this isn't defined for !PPC64 at all, or you should just use 
> target_ulong and not
> do any ifdeffing at all.
>
> r~

The helper receives i64 because it's also used by Vector Insert From VSR 
in patch 17. We can drop the ifdef and always tcg_gen_extu_tl_i64 though.

--
Matheus K. Ferst
Instituto de Pesquisas ELDORADO <http://www.eldorado.org.br/>
Analista de Software
Aviso Legal - Disclaimer <https://www.eldorado.org.br/disclaimer.html>