diff mbox series

[1/8] configure: Use -std=gnu11

Message ID 20210611233347.653129-2-richard.henderson@linaro.org
State Superseded
Headers show
Series configure: Change to -std=gnu11 | expand

Commit Message

Richard Henderson June 11, 2021, 11:33 p.m. UTC
From: Richard Henderson <richard.henderson@liaro.org>


Now that the minimum gcc version is 7.5, we can use C11.
This will allow lots of cleanups to the code, currently
hidden behind macros in include/qemu/compiler.h.

Signed-off-by: Richard Henderson <richard.henderson@liaro.org>

---
 configure   | 4 ++--
 meson.build | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

-- 
2.25.1

Comments

Thomas Huth June 15, 2021, 7:42 a.m. UTC | #1
On 12/06/2021 01.33, Richard Henderson wrote:
> From: Richard Henderson <richard.henderson@liaro.org>

> 

> Now that the minimum gcc version is 7.5, we can use C11.

> This will allow lots of cleanups to the code, currently

> hidden behind macros in include/qemu/compiler.h.

> 

> Signed-off-by: Richard Henderson <richard.henderson@liaro.org>

> ---

>   configure   | 4 ++--

>   meson.build | 2 +-

>   2 files changed, 3 insertions(+), 3 deletions(-)

> 

> diff --git a/configure b/configure

> index 8dcb9965b2..0489864667 100755

> --- a/configure

> +++ b/configure

> @@ -159,7 +159,7 @@ update_cxxflags() {

>       # options which some versions of GCC's C++ compiler complain about

>       # because they only make sense for C programs.

>       QEMU_CXXFLAGS="$QEMU_CXXFLAGS -D__STDC_LIMIT_MACROS -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS"

> -    CONFIGURE_CXXFLAGS=$(echo "$CONFIGURE_CFLAGS" | sed s/-std=gnu99/-std=gnu++11/)

> +    CONFIGURE_CXXFLAGS=$(echo "$CONFIGURE_CFLAGS" | sed s/-std=gnu11/-std=gnu++11/)

>       for arg in $QEMU_CFLAGS; do

>           case $arg in

>               -Wstrict-prototypes|-Wmissing-prototypes|-Wnested-externs|\

> @@ -538,7 +538,7 @@ QEMU_CFLAGS="-Wstrict-prototypes -Wredundant-decls $QEMU_CFLAGS"

>   QEMU_CFLAGS="-D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE $QEMU_CFLAGS"

>   

>   # Flags that are needed during configure but later taken care of by Meson

> -CONFIGURE_CFLAGS="-std=gnu99 -Wall"

> +CONFIGURE_CFLAGS="-std=gnu11 -Wall"

>   CONFIGURE_LDFLAGS=

>   

>   

> diff --git a/meson.build b/meson.build

> index d2a9ce91f5..c070cb6aa7 100644

> --- a/meson.build

> +++ b/meson.build

> @@ -1,5 +1,5 @@

>   project('qemu', ['c'], meson_version: '>=0.55.0',

> -        default_options: ['warning_level=1', 'c_std=gnu99', 'cpp_std=gnu++11', 'b_colorout=auto'] +

> +        default_options: ['warning_level=1', 'c_std=gnu11', 'cpp_std=gnu++11', 'b_colorout=auto'] +

>                            (meson.version().version_compare('>=0.56.0') ? [ 'b_staticpic=false' ] : []),

>           version: run_command('head', meson.source_root() / 'VERSION').stdout().strip())


Reviewed-by: Thomas Huth <thuth@redhat.com>
diff mbox series

Patch

diff --git a/configure b/configure
index 8dcb9965b2..0489864667 100755
--- a/configure
+++ b/configure
@@ -159,7 +159,7 @@  update_cxxflags() {
     # options which some versions of GCC's C++ compiler complain about
     # because they only make sense for C programs.
     QEMU_CXXFLAGS="$QEMU_CXXFLAGS -D__STDC_LIMIT_MACROS -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS"
-    CONFIGURE_CXXFLAGS=$(echo "$CONFIGURE_CFLAGS" | sed s/-std=gnu99/-std=gnu++11/)
+    CONFIGURE_CXXFLAGS=$(echo "$CONFIGURE_CFLAGS" | sed s/-std=gnu11/-std=gnu++11/)
     for arg in $QEMU_CFLAGS; do
         case $arg in
             -Wstrict-prototypes|-Wmissing-prototypes|-Wnested-externs|\
@@ -538,7 +538,7 @@  QEMU_CFLAGS="-Wstrict-prototypes -Wredundant-decls $QEMU_CFLAGS"
 QEMU_CFLAGS="-D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE $QEMU_CFLAGS"
 
 # Flags that are needed during configure but later taken care of by Meson
-CONFIGURE_CFLAGS="-std=gnu99 -Wall"
+CONFIGURE_CFLAGS="-std=gnu11 -Wall"
 CONFIGURE_LDFLAGS=
 
 
diff --git a/meson.build b/meson.build
index d2a9ce91f5..c070cb6aa7 100644
--- a/meson.build
+++ b/meson.build
@@ -1,5 +1,5 @@ 
 project('qemu', ['c'], meson_version: '>=0.55.0',
-        default_options: ['warning_level=1', 'c_std=gnu99', 'cpp_std=gnu++11', 'b_colorout=auto'] +
+        default_options: ['warning_level=1', 'c_std=gnu11', 'cpp_std=gnu++11', 'b_colorout=auto'] +
                          (meson.version().version_compare('>=0.56.0') ? [ 'b_staticpic=false' ] : []),
         version: run_command('head', meson.source_root() / 'VERSION').stdout().strip())