[lldb-dev] Enabling warnings as errors

[Eric Christopher]

Right now llvm does this just via a bot that checks for errors and leaves
it up to the developers to get it clean. It would definitely be nice to get
lldb at least clean with clang and the warning options from llvm and I'll
check in (or send out if they're less obvious) a few patches to do that
shortly.

-eric

On Thu Aug 14 2014 at 6:02:48 PM Keno Fischer <kfischer at college.harvard.edu>
wrote:

> I feel conflicted about this. On the one hand, lldb's build has
> significantly more warnings than LLVM and Clang, so it would be great
> to clean this up. On the other hand, I've been bitten by libraries
> that have this enabled in release builds because compilers introduce
> new warnings. I think I would be fine with this if it were not enabled
> in tagged releases (not sure how to do that though).
>
> On Thu, Aug 14, 2014 at 1:38 PM, Zachary Turner <zturner at google.com>
> wrote:
> > What would it take to enable warnings as errors on all platforms?
> >
> > 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.
> >
> > Thoughts?
> >
> > _______________________________________________
> > lldb-dev mailing list
> > lldb-dev at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev
> >
> _______________________________________________
> lldb-dev mailing list
> lldb-dev at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20140815/c35408d1/attachment.html>