<div dir="ltr">Just fyi, I'm planning to submit as-is for now.  We can treat disabling specific warnings all the time orthogonally, since we would need to discuss what warnings specifically etc.  In any case, in a Werror build turning everything off seems the most prudent.</div><br><div class="gmail_quote"><div dir="ltr">On Tue, Oct 4, 2016 at 1:20 PM Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">zturner added a comment.<br class="gmail_msg">
<br class="gmail_msg">
In <a href="https://reviews.llvm.org/D25246#561323" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D25246#561323</a>, @Eugene.Zelenko wrote:<br class="gmail_msg">
<br class="gmail_msg">
> You could use -Wno-error=<warning> instead.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
Yes but it's a little bit annoying to track down every single one we get in this file with all the different compilers.  Since we can't really control the generated code, there's not much we can do.  Seems easier to just turn them off.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D25246" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D25246</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>