<div dir="ltr">In the clang/llvm world, I often get a small handful of warnings when building locally, almost always due to my local compiler having buggy warnings. If -Werror was enabled by default in the clang/llvm build, I'd have a broken build fairly often.<div>
<br></div><div>Maybe we could detect compiler versions and only enable -Werror for compilers that are in some whitelist.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Aug 14, 2014 at 5:42 PM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">There are alot of compilers, but only a small number that we support building with.  And really, the burden would only be on making sure that the compiler you personally build with builds warning free (which, unfortunately, even that doesn't always happen).  If someone checks something in on Mac that introduces warnings on Windows with MSVC, then a Windows person would need to fix it, but I don't think that's too big of a deal (well, I don't mind anyway).  The difference is just that "strive" would become "require".  Sometimes you can't fix warnings for whatever reason, but you could still disable them in that case.<div>

<br></div><div>I guess in an ideal world I would agree with you, that we should strive to keep warnings down to as low a level as possible.  But in practice I think it's too easy to ignore them, and so people do.</div>

</div><div class="gmail_extra"><br><br><div class="gmail_quote"><div class="">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>

</div><div class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">There are so many compilers and each compiler is good at detecting things that other compilers don't warn about.<br>


<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.<br>
<span><font color="#888888"><br>
Greg<br>
</font></span><div><div><br>
> On Aug 14, 2014, at 10:38 AM, Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>> wrote:<br>
><br>
> What would it take to enable warnings as errors on all platforms?<br>
><br>
> I've done alot of work getting the windows build warning free, but it seems like very time I sync the code on linux or Mac, there's many many new warnings introduced.  I'm always vigilant about fixing warnings in my own code, it would be nice if everyone else had the same level of vigilance.  If warnings broke the build, then I think that would be a good motivator to fix your warnings.<br>


><br>
> Thoughts?<br>
</div></div><div><div>> _______________________________________________<br>
> lldb-dev mailing list<br>
> <a href="mailto:lldb-dev@cs.uiuc.edu" target="_blank">lldb-dev@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev</a><br>
<br>
</div></div></blockquote></div></div><br></div>
<br>_______________________________________________<br>
lldb-dev mailing list<br>
<a href="mailto:lldb-dev@cs.uiuc.edu">lldb-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev</a><br>
<br></blockquote></div><br></div>