Re: [PATCH] Remove the CONFIG_PRAGMA_DIAGNOSTIC_AVAILABLE switch

2020-07-09 Thread Thomas Huth
On 09/07/2020 12.51, Stefan Hajnoczi wrote: > On Thu, Jul 09, 2020 at 07:34:56AM +0200, Thomas Huth wrote: >> diff --git a/util/coroutine-ucontext.c b/util/coroutine-ucontext.c >> index f0b66320e1..a4e6446ed9 100644 >> --- a/util/coroutine-ucontext.c >> +++ b/util/coroutine-ucontext.c >> @@ -237,19

Re: [PATCH] Remove the CONFIG_PRAGMA_DIAGNOSTIC_AVAILABLE switch

2020-07-09 Thread Stefan Hajnoczi
On Thu, Jul 09, 2020 at 07:34:56AM +0200, Thomas Huth wrote: > diff --git a/util/coroutine-ucontext.c b/util/coroutine-ucontext.c > index f0b66320e1..a4e6446ed9 100644 > --- a/util/coroutine-ucontext.c > +++ b/util/coroutine-ucontext.c > @@ -237,19 +237,15 @@ Coroutine *qemu_coroutine_new(void) >

Re: [PATCH] Remove the CONFIG_PRAGMA_DIAGNOSTIC_AVAILABLE switch

2020-07-09 Thread Paolo Bonzini
On 09/07/20 07:34, Thomas Huth wrote: > GCC supports "#pragma GCC diagnostic" since version 4.6, and > Clang seems to support it, too, since its early versions 3.x. > That means that our minimum required compiler versions all support > this pragma already and we can remove the test from configure a

Re: [PATCH] Remove the CONFIG_PRAGMA_DIAGNOSTIC_AVAILABLE switch

2020-07-09 Thread Daniel P . Berrangé
On Thu, Jul 09, 2020 at 07:34:56AM +0200, Thomas Huth wrote: > GCC supports "#pragma GCC diagnostic" since version 4.6, and > Clang seems to support it, too, since its early versions 3.x. > That means that our minimum required compiler versions all support > this pragma already and we can remove th

[PATCH] Remove the CONFIG_PRAGMA_DIAGNOSTIC_AVAILABLE switch

2020-07-08 Thread Thomas Huth
GCC supports "#pragma GCC diagnostic" since version 4.6, and Clang seems to support it, too, since its early versions 3.x. That means that our minimum required compiler versions all support this pragma already and we can remove the test from configure and all the related #ifdefs in the code. Signe