[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: |
Aurelien Jarno |
Subject: |
Re: [Qemu-devel] [PATCH] Fix a number of unused-but-set-variable warnings (new with gcc-4.6) |
Date: |
Tue, 17 May 2011 19:51:13 +0200 |
User-agent: |
Mutt/1.5.20 (2009-06-14) |
On Tue, May 17, 2011 at 01:25:10PM +0200, Christophe Fergeau wrote:
> Hi Hans,
>
> On Tue, May 03, 2011 at 01:03:40PM +0200, Hans de Goede wrote:
> > diff --git a/target-i386/kvm.c b/target-i386/kvm.c
> > index a13599d..e9e8d54 100644
> > --- a/target-i386/kvm.c
> > +++ b/target-i386/kvm.c
> > @@ -950,7 +950,7 @@ static int kvm_get_xsave(CPUState *env)
> > @@ -966,7 +966,7 @@ static int kvm_get_xsave(CPUState *env)
> > cwd = (uint16_t)xsave->region[0];
> > swd = (uint16_t)(xsave->region[0] >> 16);
> > twd = (uint16_t)xsave->region[1];
> > - fop = (uint16_t)(xsave->region[1] >> 16);
> > + /* fop = (uint16_t)(xsave->region[1] >> 16); */
>
> Wouldn't it be better to drop this line?
>
> > env->fpstt = (swd >> 11) & 7;
> > env->fpus = swd;
> > env->fpuc = cwd;
> > diff --git a/tcg/tcg.c b/tcg/tcg.c
> > index 8748c05..11a8daf 100644
> > --- 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
>
> This function uses #if defined(TCG_TARGET_I386) in other places, so I'd use
> parentheses here for consistency.
>
> > 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
>
> Same here.
>
> Apart from that, I also need this patch to be able to build qemu on fedora
> 15 :)
>
You should also probably split the patch in two parts, as it touch
totally different parts of QEMU, and thus is going to be
reviewed/applied by different person. I am fine with the TCG part after
the comments from Christophe, so I'll apply it as soon as the new
version is out.
--
Aurelien Jarno GPG: 1024D/F1BCDB73
address@hidden http://www.aurel32.net
- [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),
Aurelien Jarno <=
- Re: [Qemu-devel] [PATCH] Fix a number of unused-but-set-variable warnings (new with gcc-4.6), Amit Shah, 2011/05/27
- [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