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

Aaron Ballman via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 25 06:21:49 PST 2016

aaron.ballman added a comment.

LGTM again, so I have commit as r258687.

Comment at: C:/LLVM/llvm/cmake/modules/HandleLLVMOptions.cmake:366
@@ -365,1 +365,3 @@
+  # /Zc:strictStrings is incompatible with VS12's (Visual Studio 2013's)
+  # debug mode headers. Instead of only enabling them in VS2013's debug mode,
> I presume it is not forward compatible. I'm also not sure how I'd enable strictStrings in a forwards compatible way.

I think that the way you've done it here is the best we can do.


More information about the llvm-commits mailing list