diff options
| author | Emilia Kasper <emilia@openssl.org> | 2015-04-14 16:04:40 +0200 |
|---|---|---|
| committer | Emilia Kasper <emilia@openssl.org> | 2015-04-17 18:17:50 +0200 |
| commit | 13efe9d17e7ee522c5aaa07f3076184161ede61f (patch) | |
| tree | 172f162c79b1cf4ccb0d2d71e75b430c99b76ae4 /Configurations | |
| parent | SunOS non-posix shells do not grok export name=value (diff) | |
| download | openssl-13efe9d17e7ee522c5aaa07f3076184161ede61f.tar.xz openssl-13efe9d17e7ee522c5aaa07f3076184161ede61f.zip | |
Use -Wall -Wextra with clang
The disabled set of -Weverything is hard to maintain across versions.
Use -Wall -Wextra but also document other useful warnings that currently trigger.
Reviewed-by: Rich Salz <rsalz@openssl.org>
Diffstat (limited to 'Configurations')
| -rw-r--r-- | Configurations/10-main.conf | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/Configurations/10-main.conf b/Configurations/10-main.conf index ab269ba4d9..aa4c76acfa 100644 --- a/Configurations/10-main.conf +++ b/Configurations/10-main.conf @@ -641,7 +641,14 @@ "linux-x86_64-clang" => { inherit_from => [ "linux-x86_64" ], cc => "clang", - cflags => "-m64 -DL_ENDIAN -Weverything $clang_disabled_warnings -Qunused-arguments", + # TODO(openssl-team): fix problems and investigate if (at least) the + # following warnings can also be enabled: + # -Wconditional-uninitialized, -Wswitch-enum, -Wunused-macros, + # -Wmissing-field-initializers, -Wmissing-variable-declarations, + # -Wincompatible-pointer-types-discards-qualifiers, -Wcast-align, + # -Wunreachable-code -Wunused-parameter -Wlanguage-extension-token + # -Wextended-offsetof + cflags => "-m64 -DL_ENDIAN -Wall -Wextra $clang_disabled_warnings -Qunused-arguments", }, "linux-x32" => { inherit_from => [ "linux-generic32", asm("x86_64_asm") ], |