[LLVMdev] Enabling level 4 warnings by default for MSVC builds

[Kaylor, Andrew]

Following my recent round of changes to disable selected MSVC builds both the core llvm project and clang are building cleanly with MSVC 2013 Update 4 with the LLVM_ENABLE_WARNINGS option set to 'true' (enabling MSVC's level 4 warnings).  I have not built any other projects with MSVC, but I would expect at most a manageable, and possibly even useful, set of warnings to be produced.

The LLVM_ENABLE_WARNINGS option currently defaults to 'false' for MSVC builds due to the previously extremely high number of warnings produced.  Now that the warnings that were judged to fall below the line of significance of have been disabled, I'd like to reverse the default to match builds with other compilers.

Does anyone object to doing so?

Thanks,
Andy

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20150403/d2f4eabd/attachment.html>