[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 07/21] target-arm: A64: add support for 3 src da
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 07/21] target-arm: A64: add support for 3 src data proc insns |
Date: |
Fri, 20 Dec 2013 13:18:52 +0000 |
On 19 December 2013 19:29, Richard Henderson <address@hidden> wrote:
> On 12/17/2013 07:12 AM, Peter Maydell wrote:
>> + tcg_gen_mul_i64(tcg_tmp, tcg_op1, tcg_op2);
>> + if (is_sub) {
>> + tcg_gen_sub_i64(cpu_reg(s, rd), cpu_reg(s, ra), tcg_tmp);
>> + } else {
>> + tcg_gen_add_i64(cpu_reg(s, rd), cpu_reg(s, ra), tcg_tmp);
>> + }
>
> Perhaps worth noticing the RA=XZR special case for the MUL alias?
Yeah, makes sense: have adjusted to:
+ if (ra == 31) {
+ /* We special-case rA == XZR as it is the standard MUL alias */
+ tcg_gen_mul_i64(cpu_reg(s, rd), tcg_op1, tcg_op2);
+ } else {
+ tcg_gen_mul_i64(tcg_tmp, tcg_op1, tcg_op2);
+ if (is_sub) {
+ tcg_gen_sub_i64(cpu_reg(s, rd), cpu_reg(s, ra), tcg_tmp);
+ } else {
+ tcg_gen_add_i64(cpu_reg(s, rd), cpu_reg(s, ra), tcg_tmp);
+ }
+ }
thanks
-- PMM
[Qemu-devel] [PATCH 18/21] target-arm: aarch64: add support for ld lit, Peter Maydell, 2013/12/17
[Qemu-devel] [PATCH 14/21] target-arm: A64: Implement minimal set of EL0-visible sysregs, Peter Maydell, 2013/12/17
[Qemu-devel] [PATCH 02/21] target-arm: A64: add support for ld/st unsigned imm, Peter Maydell, 2013/12/17