[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 58/65] target-sparc: Use ctpop helper
From: |
Mark Cave-Ayland |
Subject: |
Re: [Qemu-devel] [PATCH 58/65] target-sparc: Use ctpop helper |
Date: |
Fri, 30 Dec 2016 18:25:39 +0000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Icedove/45.5.1 |
On 24/12/16 04:00, Richard Henderson wrote:
> Cc: Mark Cave-Ayland <address@hidden>
> Signed-off-by: Richard Henderson <address@hidden>
> ---
> target/sparc/helper.c | 5 -----
> target/sparc/helper.h | 1 -
> target/sparc/translate.c | 2 +-
> 3 files changed, 1 insertion(+), 7 deletions(-)
>
> diff --git a/target/sparc/helper.c b/target/sparc/helper.c
> index 359b0b1..1d85489 100644
> --- a/target/sparc/helper.c
> +++ b/target/sparc/helper.c
> @@ -49,11 +49,6 @@ void helper_debug(CPUSPARCState *env)
> }
>
> #ifdef TARGET_SPARC64
> -target_ulong helper_popc(target_ulong val)
> -{
> - return ctpop64(val);
> -}
> -
> void helper_tick_set_count(void *opaque, uint64_t count)
> {
> #if !defined(CONFIG_USER_ONLY)
> diff --git a/target/sparc/helper.h b/target/sparc/helper.h
> index 0cf1bfb..3ef38b9 100644
> --- a/target/sparc/helper.h
> +++ b/target/sparc/helper.h
> @@ -16,7 +16,6 @@ DEF_HELPER_2(wrccr, void, env, tl)
> DEF_HELPER_1(rdcwp, tl, env)
> DEF_HELPER_2(wrcwp, void, env, tl)
> DEF_HELPER_FLAGS_2(array8, TCG_CALL_NO_RWG_SE, tl, tl, tl)
> -DEF_HELPER_FLAGS_1(popc, TCG_CALL_NO_RWG_SE, tl, tl)
> DEF_HELPER_FLAGS_2(set_softint, TCG_CALL_NO_RWG, void, env, i64)
> DEF_HELPER_FLAGS_2(clear_softint, TCG_CALL_NO_RWG, void, env, i64)
> DEF_HELPER_FLAGS_2(write_softint, TCG_CALL_NO_RWG, void, env, i64)
> diff --git a/target/sparc/translate.c b/target/sparc/translate.c
> index 2205f89..ead585e 100644
> --- a/target/sparc/translate.c
> +++ b/target/sparc/translate.c
> @@ -4647,7 +4647,7 @@ static void disas_sparc_insn(DisasContext * dc,
> unsigned int insn)
> gen_store_gpr(dc, rd, cpu_dst);
> break;
> case 0x2e: /* V9 popc */
> - gen_helper_popc(cpu_dst, cpu_src2);
> + tcg_gen_ctpop_tl(cpu_dst, cpu_src2);
> gen_store_gpr(dc, rd, cpu_dst);
> break;
> case 0x2f: /* V9 movr */
>
Looks good to me.
Acked-by: Mark Cave-Ayland <address@hidden>
ATB,
Mark.
- [Qemu-devel] [PATCH 46/65] tcg/i386: Hoist common arguments in tcg_out_op, (continued)
- [Qemu-devel] [PATCH 46/65] tcg/i386: Hoist common arguments in tcg_out_op, Richard Henderson, 2016/12/23
- [Qemu-devel] [PATCH 48/65] tcg/i386: Handle ctz and clz opcodes, Richard Henderson, 2016/12/23
- [Qemu-devel] [PATCH 51/65] target-arm: Use clrsb helper, Richard Henderson, 2016/12/23
- [Qemu-devel] [PATCH 55/65] target-alpha: Use ctpop helper, Richard Henderson, 2016/12/23
- [Qemu-devel] [PATCH 57/65] target-s390x: Avoid a loop for popcnt, Richard Henderson, 2016/12/23
- [Qemu-devel] [PATCH 54/65] tcg: Add opcode for ctpop, Richard Henderson, 2016/12/23
- [Qemu-devel] [PATCH 61/65] qemu/host-utils.h: Reduce the operation count in the fallback ctpop, Richard Henderson, 2016/12/23
- [Qemu-devel] [PATCH 60/65] target-i386: Use ctpop helper, Richard Henderson, 2016/12/23
- [Qemu-devel] [PATCH 49/65] tcg/i386: Rely on undefined/undocumented behaviour of BSF/BSR, Richard Henderson, 2016/12/23
- [Qemu-devel] [PATCH 58/65] target-sparc: Use ctpop helper, Richard Henderson, 2016/12/23
- Re: [Qemu-devel] [PATCH 58/65] target-sparc: Use ctpop helper,
Mark Cave-Ayland <=
- [Qemu-devel] [PATCH 65/65] tcg/i386: Handle ctpop opcode, Richard Henderson, 2016/12/23
- [Qemu-devel] [PATCH 62/65] tests: New test-bitcnt, Richard Henderson, 2016/12/23
- [Qemu-devel] [PATCH 63/65] tcg: Use ctpop to generate ctz if needed, Richard Henderson, 2016/12/23
- [Qemu-devel] [PATCH 64/65] tcg/ppc: Handle ctpop opcode, Richard Henderson, 2016/12/23