diff mbox

tcg/arm: Fix compiler error caused by unused function

Message ID 1311970049-5472-1-git-send-email-weil@mail.berlios.de
State Superseded
Headers show

Commit Message

Stefan Weil July 29, 2011, 8:07 p.m. UTC
Function tcg_out_addi is obviously unused and causes a compiler
error (tested with cross compilation on x86 Debian Linux):

tcg/arm/tcg-target.c:1824: error: ‘tcg_out_addi’ defined but not used

Don't remove it because it might be useful in the future,
but declare it inline. This fixes the compiler error and
is similar to other tcg targets.

Cc: Andrzej Zaborowski <balrogg@gmail.com>
Signed-off-by: Stefan Weil <weil@mail.berlios.de>
---
 tcg/arm/tcg-target.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

Comments

Stefan Weil Aug. 13, 2011, 9:16 a.m. UTC | #1
Am 29.07.2011 22:07, schrieb Stefan Weil:
> Function tcg_out_addi is obviously unused and causes a compiler
> error (tested with cross compilation on x86 Debian Linux):
>
> tcg/arm/tcg-target.c:1824: error: ‘tcg_out_addi’ defined but not used
>
> Don't remove it because it might be useful in the future,
> but declare it inline. This fixes the compiler error and
> is similar to other tcg targets.
>
> Cc: Andrzej Zaborowski<balrogg@gmail.com>
> Signed-off-by: Stefan Weil<weil@mail.berlios.de>
> ---
>   tcg/arm/tcg-target.c |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/tcg/arm/tcg-target.c b/tcg/arm/tcg-target.c
> index 93eb0f1..c94a354 100644
> --- a/tcg/arm/tcg-target.c
> +++ b/tcg/arm/tcg-target.c
> @@ -1820,7 +1820,7 @@ static inline void tcg_out_st(TCGContext *s, TCGType type, int arg,
>       tcg_out_st32(s, COND_AL, arg, arg1, arg2);
>   }
>
> -static void tcg_out_addi(TCGContext *s, int reg, tcg_target_long val)
> +static inline void tcg_out_addi(TCGContext *s, int reg, tcg_target_long val)
>   {
>       if (val>  0)
>           if (val<  0x100)
>    

Ping?
Andreas Färber Aug. 24, 2011, 8:25 p.m. UTC | #2
Am 13.08.2011 um 11:16 schrieb Stefan Weil:

> Am 29.07.2011 22:07, schrieb Stefan Weil:
>> Function tcg_out_addi is obviously unused and causes a compiler
>> error (tested with cross compilation on x86 Debian Linux):
>>
>> tcg/arm/tcg-target.c:1824: error: ‘tcg_out_addi’ defined but not used
>>
>> Don't remove it because it might be useful in the future,
>> but declare it inline. This fixes the compiler error and
>> is similar to other tcg targets.
>>
>> Cc: Andrzej Zaborowski<balrogg@gmail.com>
>> Signed-off-by: Stefan Weil<weil@mail.berlios.de>
>> ---
>>  tcg/arm/tcg-target.c |    2 +-
>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/tcg/arm/tcg-target.c b/tcg/arm/tcg-target.c
>> index 93eb0f1..c94a354 100644
>> --- a/tcg/arm/tcg-target.c
>> +++ b/tcg/arm/tcg-target.c
>> @@ -1820,7 +1820,7 @@ static inline void tcg_out_st(TCGContext *s,  
>> TCGType type, int arg,
>>      tcg_out_st32(s, COND_AL, arg, arg1, arg2);
>>  }
>>
>> -static void tcg_out_addi(TCGContext *s, int reg, tcg_target_long  
>> val)
>> +static inline void tcg_out_addi(TCGContext *s, int reg,  
>> tcg_target_long val)
>>  {
>>      if (val>  0)
>>          if (val<  0x100)
>>
>
> Ping?

I don't like that solution, it's not obvious that 'inline' should be  
reverted later.

GCC has __attribute__((unused)) for that purpose. It can be prepended  
on a line of its own so that it can be removed without touching the  
actual declaration.

Andreas
diff mbox

Patch

diff --git a/tcg/arm/tcg-target.c b/tcg/arm/tcg-target.c
index 93eb0f1..c94a354 100644
--- a/tcg/arm/tcg-target.c
+++ b/tcg/arm/tcg-target.c
@@ -1820,7 +1820,7 @@  static inline void tcg_out_st(TCGContext *s, TCGType type, int arg,
     tcg_out_st32(s, COND_AL, arg, arg1, arg2);
 }
 
-static void tcg_out_addi(TCGContext *s, int reg, tcg_target_long val)
+static inline void tcg_out_addi(TCGContext *s, int reg, tcg_target_long val)
 {
     if (val > 0)
         if (val < 0x100)