<div class="gmail_quote">On Sun, Jan 15, 2012 at 5:23 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Sun, Jan 15, 2012 at 1:03 AM, Chandler Carruth <<a href="mailto:chandlerc@gmail.com">chandlerc@gmail.com</a>> wrote:<br>
> Author: chandlerc<br>
> Date: Sun Jan 15 03:03:45 2012<br>
> New Revision: 148212<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=148212&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=148212&view=rev</a><br>
> Log:<br>
> Two variables had been added for an assert, but their values were<br>
> re-computed rather than the variables be re-used just after the assert.<br>
> Just use the variables since we have them already. Fixes an unused<br>
> variable warning.<br>
<br>
</div>Thanks Chandler - attached are some patches for a few other release<br>
unused variable warnings if you think they're the right fixes.</blockquote><div><br></div><div>Yes, these are fine. Feel free to commit, and similar obvious simplifications which reduce warnings. </div></div>