[PATCH] D15731: win: Pass /W4 in front of all the -wd flags.

Nico Weber via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 22 17:51:15 PST 2015

thakis updated this revision to Diff 43496.
thakis added a comment.

more direct



Index: cmake/modules/HandleLLVMOptions.cmake
--- cmake/modules/HandleLLVMOptions.cmake
+++ cmake/modules/HandleLLVMOptions.cmake
@@ -338,7 +338,10 @@
   # Enable warnings
-    append("/W4" msvc_warning_flags)
+    # Put /W4 in front of all the -we flags. cl.exe doesn't care, but for
+    # clang-cl having /W4 after the -we flags will re-enable the warnings
+    # disabled by -we.
+    set(msvc_warning_flags "/W4 ${msvc_warning_flags}")
     # CMake appends /W3 by default, and having /W3 followed by /W4 will result in 
     # cl : Command line warning D9025 : overriding '/W3' with '/W4'.  Since this is
     # a command line warning and not a compiler warning, it cannot be suppressed except

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D15731.43496.patch
Type: text/x-patch
Size: 797 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151223/77b105a1/attachment.bin>

More information about the llvm-commits mailing list