This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] libgcc: Use braces instead of macro's empty body to avoid xgcc warnings.
- From: Chen Gang S <gang dot chen at sunrus dot com dot cn>
- To: Jakub Jelinek <jakub at redhat dot com>
- Cc: ian at airs dot com, Jeff Law <law at redhat dot com>, gcc-patches List <gcc-patches at gcc dot gnu dot org>, walt at tilera dot com
- Date: Sat, 31 Jan 2015 13:49:48 +0800
- Subject: Re: [PATCH] libgcc: Use braces instead of macro's empty body to avoid xgcc warnings.
- Authentication-results: sourceware.org; auth=none
- References: <54CC07AA dot 4090006 at sunrus dot com dot cn> <20150130225747 dot GI1746 at tucnak dot redhat dot com>
On 1/31/15 06:57, Jakub Jelinek wrote:
> On Sat, Jan 31, 2015 at 06:37:30AM +0800, Chen Gang S wrote:
>> The related warning (cross compile tile with --disable-threads):
>>
>> ../../../../gcc-tile-new/libgcc/libgcov-interface.c: In function '__gcov_fork':
>> ../../../../gcc-tile-new/libgcc/libgcov-interface.c:182:53: warning: suggest braces around empty body in an 'if' statement [-Wempty-body]
>> __GTHREAD_MUTEX_INIT_FUNCTION (&__gcov_flush_mx);
>> ^
>
> Then you should use do {} while (0) instead of {} . Also, braces not breaces.
>
OK, thanks, I shall send patch v2 within today (but maybe at night).
>> 2015-01-31 Chen Gang <gang.chen.5i5j@gmail.com>
>>
>> * gthr-single.h (__GTHREAD_MUTEX_INIT_FUNCTION): Use breaces
>> instead of macro's empty body to avoid xgcc warnings.
>> ---
>> libgcc/gthr-single.h | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/libgcc/gthr-single.h b/libgcc/gthr-single.h
>> index f084fe2..586fec5 100644
>> --- a/libgcc/gthr-single.h
>> +++ b/libgcc/gthr-single.h
>> @@ -35,7 +35,7 @@ typedef int __gthread_recursive_mutex_t;
>>
>> #define __GTHREAD_ONCE_INIT 0
>> #define __GTHREAD_MUTEX_INIT 0
>> -#define __GTHREAD_MUTEX_INIT_FUNCTION(mx)
>> +#define __GTHREAD_MUTEX_INIT_FUNCTION(mx) {}
>> #define __GTHREAD_RECURSIVE_MUTEX_INIT 0
>>
>> #define UNUSED __attribute__((unused))
>> --
>> 1.9.3
>
> Jakub
>
--
Chen Gang
Open, share, and attitude like air, water, and life which God blessed