[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] Fix a number of unused-but-set-variable warning
From: |
Amit Shah |
Subject: |
Re: [Qemu-devel] [PATCH] Fix a number of unused-but-set-variable warnings (new with gcc-4.6) |
Date: |
Fri, 27 May 2011 16:04:58 +0530 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On (Tue) 03 May 2011 [13:03:40], Hans de Goede wrote:
> ---
> target-i386/kvm.c | 4 ++--
> tcg/tcg.c | 4 ++--
> 2 files changed, 4 insertions(+), 4 deletions(-)
Thanks; just got hit by this.
However, there are a couple of whitespace issues:
> --- a/tcg/tcg.c
> +++ b/tcg/tcg.c
> @@ -585,7 +585,7 @@ void tcg_register_helper(void *func, const char *name)
> void tcg_gen_callN(TCGContext *s, TCGv_ptr func, unsigned int flags,
> int sizemask, TCGArg ret, int nargs, TCGArg *args)
> {
> -#ifdef TCG_TARGET_I386
> +#if defined TCG_TARGET_I386 && TCG_TARGET_REG_BITS < 64
> int call_type;
> #endif
> int i;
> @@ -612,7 +612,7 @@ void tcg_gen_callN(TCGContext *s, TCGv_ptr func, unsigned
> int flags,
>
> *gen_opc_ptr++ = INDEX_op_call;
> nparam = gen_opparam_ptr++;
> -#ifdef TCG_TARGET_I386
> +#if defined TCG_TARGET_I386 && TCG_TARGET_REG_BITS < 64
> call_type = (flags & TCG_CALL_TYPE_MASK);
> #endif
> if (ret != TCG_CALL_DUMMY_ARG) {
Both these lines have a trailing space.
Care to resubmit with Anthony in CC? You can add:
Acked-by: Amit Shah <address@hidden>
Amit
- [Qemu-devel] [PATCH] Fix a number of unused-but-set-variable warnings (new with gcc-4.6), Hans de Goede, 2011/05/03
- Re: [Qemu-devel] [PATCH] Fix a number of unused-but-set-variable warnings (new with gcc-4.6), Christophe Fergeau, 2011/05/17
- Re: [Qemu-devel] [PATCH] Fix a number of unused-but-set-variable warnings (new with gcc-4.6),
Amit Shah <=
- [Qemu-devel] [PATCH 0/2] Fix unused-but-set-variable warnings, Christophe Fergeau, 2011/05/30
- [Qemu-devel] [PATCH 1/2] tcg: Fix unused-but-set-variable warning, Christophe Fergeau, 2011/05/30
- [Qemu-devel] [PATCH 2/2] kvm: Fix unused-but-set-variable warning, Christophe Fergeau, 2011/05/30
- Re: [Qemu-devel] [PATCH 0/2] Fix unused-but-set-variable warnings, Amit Shah, 2011/05/30
- Re: [Qemu-devel] [PATCH 0/2] Fix unused-but-set-variable warnings, Christophe Fergeau, 2011/05/30
- Re: [Qemu-devel] [PATCH 0/2] Fix unused-but-set-variable warnings, Amit Shah, 2011/05/31
- [Qemu-devel] [PATCH-v3 0/2] Fix unused-but-set-variable warnings, Christophe Fergeau, 2011/05/31
- [Qemu-devel] [PATCH-v3 1/2] tcg: Fix unused-but-set-variable warning, Christophe Fergeau, 2011/05/31
- [Qemu-devel] [PATCH-v3 2/2] kvm: Fix unused-but-set-variable warning, Christophe Fergeau, 2011/05/31