<div class="gmail_quote">On Thu, May 26, 2011 at 4:16 PM, Devang Patel <span dir="ltr"><<a href="mailto:dpatel@apple.com">dpatel@apple.com</a>></span> wrote:</div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div id=":1ix">+      if (!VI->isTerminator() &&<br>
+          DVI->getParent() != VI->getParent() || DT->dominates(DVI, VI)) {</div></blockquote></div><br><div>This will ignore (!VI->isTerminator()) if DT->dominates(DVI, VI returns true. ;] We have a warning on this construct in Clang (based on GCC's warning). I added parentheses in r132165 to more closely match the intent your commit log conveyed, but let me know if that isn't correct.</div>