Re: [Xen-devel] [PATCH] clang: disable the gcc-compat warnings for read_atomic

2017-08-01 Thread Roger Pau Monne
On Tue, Apr 11, 2017 at 08:54:20AM +0100, Roger Pau Monne wrote: > clang gcc-compat warnings can wrongly fire when certain constructions are > used, > at least the following flow: > > switch ( ... ) > { > case ...: > while ( ({ int x; switch ( foo ) { case 1: x = 1; break; } x }) ) > { >

Re: [Xen-devel] [PATCH] clang: disable the gcc-compat warnings for read_atomic

2017-04-13 Thread Julien Grall
Hi Jan, On 13/04/17 16:11, Jan Beulich wrote: On 11.04.17 at 09:54, wrote: clang gcc-compat warnings can wrongly fire when certain constructions are used, at least the following flow: switch ( ... ) { case ...: while ( ({ int x; switch ( foo ) { case 1: x = 1; break; } x }) ) {

Re: [Xen-devel] [PATCH] clang: disable the gcc-compat warnings for read_atomic

2017-04-13 Thread Jan Beulich
>>> On 11.04.17 at 09:54, wrote: > clang gcc-compat warnings can wrongly fire when certain constructions are > used, > at least the following flow: > > switch ( ... ) > { > case ...: > while ( ({ int x; switch ( foo ) { case 1: x = 1; break; } x }) ) > { > ... > > Will cause cla

Re: [Xen-devel] [PATCH] clang: disable the gcc-compat warnings for read_atomic

2017-04-11 Thread Roger Pau Monne
On Tue, Apr 11, 2017 at 02:35:59AM -0600, Jan Beulich wrote: > >>> On 11.04.17 at 09:54, wrote: > > clang gcc-compat warnings can wrongly fire when certain constructions are > > used, > > at least the following flow: > > > > switch ( ... ) > > { > > case ...: > > while ( ({ int x; switch ( f

Re: [Xen-devel] [PATCH] clang: disable the gcc-compat warnings for read_atomic

2017-04-11 Thread Jan Beulich
>>> On 11.04.17 at 09:54, wrote: > clang gcc-compat warnings can wrongly fire when certain constructions are > used, > at least the following flow: > > switch ( ... ) > { > case ...: > while ( ({ int x; switch ( foo ) { case 1: x = 1; break; } x }) ) > { > ... > > Will cause cla