meson: remove -std=c++11 from intel/tools
authorDylan Baker <dylan@pnwbakers.com>
Tue, 29 Jan 2019 19:25:30 +0000 (11:25 -0800)
committerDylan Baker <dylan@pnwbakers.com>
Thu, 31 Jan 2019 21:42:16 +0000 (21:42 +0000)
for meson all C++ code is already compiled as C++11, so it's
unnecessary. It's also the wrong way to do this, if we really needed
this the correct way is to set:

```meson
executable(
  ...
  override_options : ['cpp_std=c++11'],
)
```

Which ensures not only that the correct syntax for the current
compiler is used, but also that meson doesn't create arguments like
`-std=c++14 ... -std=c++11`

Reviewed-by: Caio Marcelo de Oliveira Filho <caio.oliveira@intel.com>
Reviewed-by: Eric Engestrom <eric.engestrom@intel.com>
src/intel/tools/meson.build

index 02ebb685531f6c32ff432038a70b626f6dc80eec..2ef49ddc17c240b1022f07278354a072e62bd7e5 100644 (file)
@@ -123,7 +123,7 @@ if with_tools.contains('intel-ui')
     include_directories : [inc_common, inc_intel],
     link_with : [libintel_common, libintel_compiler, libintel_dev, libmesa_util, libaub],
     c_args : [c_vis_args, no_override_init_args],
-    cpp_args : ['-fpermissive', '-std=c++11', '-Wno-parentheses', '-Wno-class-memaccess'],
+    cpp_args : ['-fpermissive', '-Wno-parentheses', '-Wno-class-memaccess'],
     install : true
   )
 endif