<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Apr 3, 2015 at 1:37 PM, Kaylor, Andrew <span dir="ltr"><<a href="mailto:andrew.kaylor@intel.com" target="_blank">andrew.kaylor@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





<div lang="EN-US" link="blue" vlink="purple">
<div>
<p class="MsoNormal">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.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">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.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Does anyone object to doing so?</p></div></div></blockquote><div><br></div><div>Please do! I didn't realize the default value was based on usage of MSVC. I've been enabling it manually for so long I'd forgotten I had to.</div></div></div></div>