Message ID | 1311970049-5472-1-git-send-email-weil@mail.berlios.de |
---|---|
State | Superseded |
Headers | show |
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?
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 --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)
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(-)