<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Aug 14, 2014 at 5:28 PM, Greg Clayton <span dir="ltr"><<a href="mailto:gclayton@apple.com" target="_blank">gclayton@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":h1r" class="a3s" style="overflow:hidden">There are so many compilers and each compiler is good at detecting things that other compilers don't warn about.<br>
</div></blockquote><div><br></div><div>You can easily disable warnings that don't have value. We do this all the time in LLVM.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div id=":h1r" class="a3s" style="overflow:hidden">
<br>
So I don't think we should enable warnings as errors, but I do think we should strive to keep warnings down to as low a level as possible.</div></blockquote></div><br></div><div class="gmail_extra">I think there should be a WERROR mode in the CMake build, and build bots should run with it. That way releases etc aren't impacted by new warnings, but developers *do* notice when they introduce a warning.</div>
</div>