[PATCH] D15784: Enable 2 warnings on MSVC, turn on StringPooling & intrinsic functions

Aaron Ballman via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 27 06:38:27 PST 2015

aaron.ballman added a comment.

The /Zc options are good to enable -- more conformance is better. I have no opinions on /Oi, and some questions about /GF.

Comment at: cmake/modules/HandleLLVMOptions.cmake:369
@@ +368,3 @@
+  # Eliminate Duplicate Strings
I think this one is reasonable, but I worry about it requiring use of /bigobj. What are the size differences for our executables with this option disabled vs enabled?

Comment at: cmake/modules/HandleLLVMOptions.cmake:375
@@ +374,3 @@
+  # Enforce type conversion rules
+  append("/Zc:rvalueCast" CMAKE_C_FLAGS CMAKE_CXX_FLAGS)
Should this be specified for CMAKE_C_FLAGS as well? I know MSVC doesn't really distinguish between the C and C++ compiler, but it seems a bit strange (though likely harmless).


More information about the llvm-commits mailing list