meson: Add -Werror=gnu-empty-initializer to MSVC compat args
authorDylan Baker <dylan@pnwbakers.com>
Thu, 21 Nov 2019 17:11:45 +0000 (09:11 -0800)
committerDylan Baker <dylan@pnwbakers.com>
Tue, 26 Nov 2019 20:48:11 +0000 (12:48 -0800)
Only clang has this argument (at least as of clang 8 and gcc 9), which
errors when using the gcc empty initializer syntax in C:

```C
struct foo f = {};
```

GCC has a warning for this, but only when using -Wpedantic, which is a
lot of noise to lose useful warnings in.

Reviewed-by: Kristian H. Kristensen <hoegsberg@google.com>
Reviewed-by: Eric Engestrom <eric.engestrom@intel.com>
meson.build

index e8bec9bb3894ff854a97390f1cf120ca63d2d01e..92e45d28640c426cba7dac59b149ff2f206de8c6 100644 (file)
@@ -996,10 +996,10 @@ else
     c_vis_args += '-fvisibility=hidden'
   endif
 
-  # Check for C and C++ arguments for MSVC2013 compatibility. These are only
-  # used in parts of the mesa code base that need to compile with old versions
-  # of MSVC, mainly common code
-  foreach a : ['-Werror=pointer-arith', '-Werror=vla']
+  # Check for C and C++ arguments for MSVC compatibility. These are only used
+  # in parts of the mesa code base that need to compile with MSVC, mainly
+  # common code
+  foreach a : ['-Werror=pointer-arith', '-Werror=vla', '-Werror=gnu-empty-initializer']
     if cc.has_argument(a)
       c_msvc_compat_args += a
     endif