[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/7] target/ppc: Use tcg_gen_gvec_dup_imm
From: |
Richard Henderson |
Subject: |
Re: [PATCH 3/7] target/ppc: Use tcg_gen_gvec_dup_imm |
Date: |
Tue, 21 Apr 2020 10:50:25 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0 |
On 4/20/20 3:34 AM, Alex Bennée wrote:
>> +GEN_VXFORM_VSPLTI(vspltisb, MO_8, 6, 12);
>> +GEN_VXFORM_VSPLTI(vspltish, MO_16, 6, 13);
>> +GEN_VXFORM_VSPLTI(vspltisw, MO_32, 6, 14);
>
> There are unused parameters opc2/opc3 parameters here.
Yes, but all of the other GEN_* macros retain them as well.
Without looking at the actual history, I'd say once upon a time they were
actually used, but we've now split opcode from implementation.
> With the removed extra unused macro params:
>
> Reviewed-by: Alex Bennée <address@hidden>
I wouldn't do that without David's approval. And possibly stripping out the
opcodes everywhere else.
r~
- Re: [PATCH 1/7] tcg: Add tcg_gen_gvec_dup_imm, (continued)
- [PATCH 4/7] target/arm: Use tcg_gen_gvec_dup_imm, Richard Henderson, 2020/04/18
- [PATCH 5/7] tcg: Use tcg_gen_gvec_dup_imm in logical simplifications, Richard Henderson, 2020/04/18
- [PATCH 6/7] tcg: Remove tcg_gen_gvec_dup{8,16,32,64}i, Richard Henderson, 2020/04/18
- Re: [PATCH 6/7] tcg: Remove tcg_gen_gvec_dup{8,16,32,64}i, LIU Zhiwei, 2020/04/20
- Re: [PATCH 6/7] tcg: Remove tcg_gen_gvec_dup{8,16,32,64}i, David Hildenbrand, 2020/04/20
- Re: [PATCH 6/7] tcg: Remove tcg_gen_gvec_dup{8,16,32,64}i, Alex Bennée, 2020/04/20