<div dir="ltr">& I see this might've been recently introduced in <span style="font-family:monospace">219741 - in that case, it's usually best to reply to the commit mail to point it out to the original author so they can clean it up.</span><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 14, 2014 at 5:02 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Is the warning emitted in a +Asserts build of LLVM? (usually such warnings don't persist long in the codebase, because so many of us build with -Werror +Asserts, so that leaves the clang warnings for -Asserts builds, but you have to be careful that you don't break the +Asserts build when you fix these (eg: the variable may be unused in -Asserts, but used in +Asserts, so removing it will fix the -Asserts warning, but break the +Asserts build))<br>
<br>
<a href="http://reviews.llvm.org/D5788" target="_blank">http://reviews.llvm.org/D5788</a><br>
<br>
<br>
</blockquote></div><br></div></div>