Thanks for the reply guys, I will fix what you pointed out. And I will work on it to improve it further.<div><br></div><div><br><div><div><br><div class="gmail_quote">On Tue, Sep 27, 2011 at 2:55 PM, Devang Patel <span dir="ltr"><<a href="mailto:dpatel@apple.com">dpatel@apple.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div style="word-wrap:break-word"><div><div></div><div class="h5"><br><div><div>On Sep 27, 2011, at 2:48 PM, David Blaikie wrote:</div>

<br><blockquote type="cite"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">2) General llvm convention is to avoid<br>
<br>
+  bool candidate = L->isThreeStageUnrollCandidate(L, LI);<br>
+  if(candidate) {<br>
….<br>
<br>
and do<br>
<br>
+  if (bool candidate = … ) {<br>
+  }<br>
<br>
if candidate is not used outside "if".<br></blockquote><div><br></div><div>At that point you might as well just write:<br><br>if (L->isThreeStageUnrollCandidate(L, LI)) {</div><div>} </div></div>
</blockquote></div><br></div></div><div>Yes, if candidate is not used inside "if".</div><div>-</div><div>Devang</div></div></blockquote></div><br></div></div></div>