On Tuesday, February 16, 2016, Tamas Berghammer <<a href="mailto:tberghammer@google.com">tberghammer@google.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I would be happy if we can keep lldb warning free but I don't think enabling -Werror is a good idea for 2 reasons:<div>* We are using a lot of different compiler and keeping the codebase warning free on all of them might not be feasible especially for the less used, older gcc versions.</div><div>* Neither llvm nor clang have -Werror enabled so if we enable it then a clang/llvm change can break our build with a warning when it is hard to justify a revert and a fix might not be trivial.</div></div></blockquote><div><br></div><div><font size="2"><span style="background-color:rgba(255,255,255,0)">Err, sorry.  I meant by default on the build bots (IIRC, some (many?) of the build bots do build with -Werror for LLVM and clang).  Yes, a new warning in clang could cause issues in LLDB, though the same thing exists for the LLVM/clang dependency.  Since this would be on the build bots, it should get resolved rather quickly.</span></font><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>In short term I would prefer to just create a policy saying everybody should write warning free code for lldb (I think it already kind of exists) and we as a community try to ensure it during code review and with fixing the possible things what slip through. In the longer term I would be happy to see -Werror turned on for llvm and clang first and then we can follow up with lldb but making this change will require a lot of discussion and might get some push back.</div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Feb 16, 2016 at 6:02 AM Saleem Abdulrasool via lldb-dev <<a href="javascript:_e(%7B%7D,'cvml','lldb-dev@lists.llvm.org');" target="_blank">lldb-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><span style="font-size:13px">Hi,</span></div><span style="font-size:13px"><div><span style="font-size:13px"><br></span></div>It seems that enabling -Werror by default is within reach for lldb now.  There currently are three warnings that remain with gcc 5.1 on Linux, and the build is clean of warnings with clang.</span><div style="font-size:13px"><br></div><div style="font-size:13px">There are two instances of type range limitations on comparisons in asserts, and one instance of string formatting which has a GNU incompatibility.<br><div><br></div><div>Is there any interest in enabling -Werror by default to help keep the build clean going forward?</div></div><div><br></div>-- <br><div>Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org</div>
</div>
_______________________________________________<br>
lldb-dev mailing list<br>
<a href="javascript:_e(%7B%7D,'cvml','lldb-dev@lists.llvm.org');" target="_blank">lldb-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev</a><br>
</blockquote></div>
</blockquote><br><br>-- <br>Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org<br>