<p dir="ltr">Hi Andy, </p>
<p dir="ltr">I think it is good idea. It would help projects that depends on LLVM as there is a lots of inline code in LLVM headers. </p>
<p dir="ltr">Do you need any help with that? I use VS quite often.</p>
<p dir="ltr">- Paweł</p>
<br><div class="gmail_quote">On Fri, Apr 3, 2015, 22:40 Kaylor, Andrew <<a href="mailto:andrew.kaylor@intel.com">andrew.kaylor@intel.com</a>> 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?<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Thanks,<u></u><u></u></p>
<p class="MsoNormal">Andy<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
</div>

______________________________<u></u>_________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu" target="_blank">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvmdev</a><br>
</blockquote></div>