Commit 751b9722 authored by Steffen Müthing's avatar Steffen Müthing

[!33] Silence clang 3.8 warning about missing braces.

Merge branch 'silence-clang3.8-missing-braces' into 'master'

ref:docker/ci This adjusts the clang 3.8 toolchains to include
-Wno-missing-braces in their flags.

Clang includes -Wmissing-braces in -Wall. This has the annoying effect of
requiring two pairs of braces when aggregate-initializing e.g. an std::array,
see [https://gitlab.dune-project.org/core/dune-common/-/jobs/79786] for an
example. Gcc did removed -Wmissing-braces from -Wall precisely for this reason
a long time ago. Clang 6 went a different route and refined the warning to not
trigger in cases such as std::array, see
[https://bugs.llvm.org/show_bug.cgi?id=21629].

See merge request [!33]

  [https://gitlab.dune-project.org/core/dune-common/-/jobs/79786]: https://gitlab.dune-project.org/core/dune-common/-/jobs/79786
  [https://bugs.llvm.org/show_bug.cgi?id=21629]: https://bugs.llvm.org/show_bug.cgi?id=21629
  [!33]: gitlab.dune-project.org/docker/ci/merge_requests/33
parents 7439409d e20aeedc
......@@ -4,7 +4,7 @@ FC=gfortran-6
CFLAGS="-Wall -fdiagnostics-color=always"
FFLAGS="-Wall -fdiagnostics-color=always"
CXXFLAGS="-Wall -fdiagnostics-color=always -std=c++14"
CXXFLAGS="-Wall -Wno-missing-braces -fdiagnostics-color=always -std=c++14"
REL_CFLAGS="-O3 -g -UNDEBUG"
REL_CXXFLAGS="-O3 -g -UNDEBUG"
......
......@@ -4,7 +4,7 @@ FC=gfortran-6
CFLAGS="-Wall -fdiagnostics-color=always"
FFLAGS="-Wall -fdiagnostics-color=always"
CXXFLAGS="-Wall -fdiagnostics-color=always -std=c++14"
CXXFLAGS="-Wall -Wno-missing-braces -fdiagnostics-color=always -std=c++14"
REL_CFLAGS="-O3 -g -DNDEBUG"
REL_CXXFLAGS="-O3 -g -DNDEBUG"
......
......@@ -4,7 +4,7 @@ FC=gfortran-6
CFLAGS="-Wall -fdiagnostics-color=always"
FFLAGS="-Wall -fdiagnostics-color=always"
CXXFLAGS="-Wall -fdiagnostics-color=always -std=c++14"
CXXFLAGS="-Wall -Wno-missing-braces -fdiagnostics-color=always -std=c++14"
REL_CFLAGS="-O3 -g -UNDEBUG"
REL_CXXFLAGS="-O3 -g -UNDEBUG"
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment