diff options
author | Orgad Shaneh <orgad.shaneh@audiocodes.com> | 2018-11-14 23:31:09 +0200 |
---|---|---|
committer | Orgad Shaneh <orgads@gmail.com> | 2018-11-15 10:01:27 +0000 |
commit | 91649cf27481a7751978695a98697281088adf8f (patch) | |
tree | 09643a4ec8546c87e07b35472312526caf098d6f /qbs/modules | |
parent | 3b30bfa35f2c1780574261c80d2a6aafd4d2650b (diff) | |
download | qt-creator-91649cf27481a7751978695a98697281088adf8f.tar.gz |
Clang: Remove some warning flags provided by LLVM
Clang unit tests produce many warnings with GCC8 and
these flags (-Wnon-virtual-dtor, -Woverloaded-virtual).
Change-Id: I54a4c9c5b2f12da3c3898fc6f3beca0d4dfe478c
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@qt.io>
Diffstat (limited to 'qbs/modules')
-rw-r--r-- | qbs/modules/libclang/functions.js | 18 |
1 files changed, 14 insertions, 4 deletions
diff --git a/qbs/modules/libclang/functions.js b/qbs/modules/libclang/functions.js index d586750b21..99577c3146 100644 --- a/qbs/modules/libclang/functions.js +++ b/qbs/modules/libclang/functions.js @@ -161,6 +161,17 @@ function toolingParameters(llvmConfig) cxxFlags: [], }; var allCxxFlags = readListOutput(llvmConfig, ["--cxxflags"]); + var badFlags = [ + "-fno-exceptions", + "/W4", + "-Wcovered-switch-default", + "-Wnon-virtual-dtor", + "-Woverloaded-virtual", + "-fPIC", + "-pedantic", + "-Wstring-conversion", + "-gsplit-dwarf" + ] for (var i = 0; i < allCxxFlags.length; ++i) { var flag = allCxxFlags[i]; if (flag.startsWith("-D") || flag.startsWith("/D")) { @@ -173,10 +184,9 @@ function toolingParameters(llvmConfig) } if (!flag.startsWith("-std") && !flag.startsWith("-O") && !flag.startsWith("/O") && !flag.startsWith("-march") - && !flag.startsWith("/EH") && flag !== "-fno-exceptions" - && flag !== "/W4" && flag !== "-Werror=date-time" - && flag !== "-Wcovered-switch-default" && flag !== "-fPIC" && flag !== "-pedantic" - && flag !== "-Wstring-conversion" && flag !== "-gsplit-dwarf") { + && !flag.startsWith("-Werror=") + && !flag.startsWith("/EH") + && !badFlags.contains(flag)) { params.cxxFlags.push(flag); } } |