Hi,

Thanks for the patch. Could you submit the patch through phabricator
<https://reviews.llvm.org> and add Greg Clayton as a reviewer. That
said, my preference would be to actually fix those warnings instead of
silencing them.

pl

On 17 October 2016 at 08:24, Justin Bogner via lldb-commits
<lldb-commits@lists.llvm.org> wrote:
> Today I checked out lldb to check that r284364 wouldn't cause any
> trouble, but there were a few warnings that fired when I tried to build
> it, breaking my -Werror build.
>
> This disables four character literal warnings (so that 'FDSC' in
> debugserver.cpp is allowed) and pedantic formatting warnings, so that
> printf-like functions can pass typed pointers to "%p" in format strings.
>
> Okay to commit?
>
>
> _______________________________________________
> lldb-commits mailing list
> lldb-commits@lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
>
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to