Re: [Qemu-devel] [PATCH] tcg/ppc/tcg-target.c: Avoid 'set but not used' gcc warnings

["Michael S. Tsirkin" <mst@redhat.com>]

On Thu, Sep 01, 2011 at 04:45:01PM +0100, Peter Maydell wrote:
> Move the declaration and initialisation of some variables in
> tcg_out_qemu_ld and tcg_out_qemu_st inside CONFIG_SOFTMMU, to
> avoid the "variable set but not used" warning of gcc 4.6.
> 
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
>  tcg/ppc/tcg-target.c |   20 ++++++++++----------
>  1 files changed, 10 insertions(+), 10 deletions(-)
> 
> diff --git a/tcg/ppc/tcg-target.c b/tcg/ppc/tcg-target.c
> index 58c8621..a003210 100644
> --- a/tcg/ppc/tcg-target.c
> +++ b/tcg/ppc/tcg-target.c
> @@ -525,14 +525,14 @@ static void *qemu_st_helpers[4] = {
>  
>  static void tcg_out_qemu_ld (TCGContext *s, const TCGArg *args, int opc)
>  {
> -    int addr_reg, data_reg, data_reg2, r0, r1, rbase, mem_index, s_bits, bswap;
> +    int addr_reg, data_reg, data_reg2, r0, r1, rbase, bswap;
>  #ifdef CONFIG_SOFTMMU
> -    int r2;
> +    int mem_index, s_bits, r2;
>      void *label1_ptr, *label2_ptr;
> -#endif
>  #if TARGET_LONG_BITS == 64
>      int addr_reg2;
>  #endif
> +#endif
>  
>      data_reg = *args++;
>      if (opc == 3)
> @@ -540,13 +540,13 @@ static void tcg_out_qemu_ld (TCGContext *s, const TCGArg *args, int opc)
>      else
>          data_reg2 = 0;
>      addr_reg = *args++;
> +
> +#ifdef CONFIG_SOFTMMU
>  #if TARGET_LONG_BITS == 64
>      addr_reg2 = *args++;
>  #endif
>      mem_index = *args;
>      s_bits = opc & 3;
> -
> -#ifdef CONFIG_SOFTMMU
>      r0 = 3;
>      r1 = 4;
>      r2 = 0;
> @@ -722,14 +722,14 @@ static void tcg_out_qemu_ld (TCGContext *s, const TCGArg *args, int opc)
>  
>  static void tcg_out_qemu_st (TCGContext *s, const TCGArg *args, int opc)
>  {
> -    int addr_reg, r0, r1, data_reg, data_reg2, mem_index, bswap, rbase;
> +    int addr_reg, r0, r1, data_reg, data_reg2, bswap, rbase;
>  #ifdef CONFIG_SOFTMMU
> -    int r2, ir;
> +    int mem_index, r2, ir;
>      void *label1_ptr, *label2_ptr;
> -#endif
>  #if TARGET_LONG_BITS == 64
>      int addr_reg2;
>  #endif
> +#endif
>  
>      data_reg = *args++;
>      if (opc == 3)
> @@ -737,12 +737,12 @@ static void tcg_out_qemu_st (TCGContext *s, const TCGArg *args, int opc)
>      else
>          data_reg2 = 0;
>      addr_reg = *args++;
> +
> +#ifdef CONFIG_SOFTMMU
>  #if TARGET_LONG_BITS == 64
>      addr_reg2 = *args++;
>  #endif
>      mem_index = *args;
> -
> -#ifdef CONFIG_SOFTMMU
>      r0 = 3;
>      r1 = 4;
>      r2 = 0;
> -- 
> 1.7.4.1
> 

No problem with the patch.  BTW, does someone care about this code
enough to refactor it fixing the ifdef spagetty?  Ideally there would be
whole functions within ifdefs.  They can be inline for efficiency.

-- 
MST