<div dir="ltr">Probably. My mistake. :)<div class="gmail_extra"><br><div class="gmail_quote">On Sun, Jan 29, 2017 at 10:44 PM, Daniel Berlin <span dir="ltr"><<a href="mailto:dberlin@dberlin.org" target="_blank">dberlin@dberlin.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I think this was one george asked me to remove parens from in the review?<div><br></div></div><div class="m_3791382838338626929HOEnZb"><div class="m_3791382838338626929h5"><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Jan 29, 2017 at 7:16 PM, Davide Italiano via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: davide<br>
Date: Sun Jan 29 21:16:43 2017<br>
New Revision: 293453<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=293453&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-pr<wbr>oject?rev=293453&view=rev</a><br>
Log:<br>
[MemorySSA] Correct an assertion surrounding with parentheses.<br>
<br>
Modified:<br>
    llvm/trunk/lib/Transforms/Util<wbr>s/MemorySSA.cpp<br>
<br>
Modified: llvm/trunk/lib/Transforms/Util<wbr>s/MemorySSA.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Utils/MemorySSA.cpp?rev=293453&r1=293452&r2=293453&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-pr<wbr>oject/llvm/trunk/lib/Transform<wbr>s/Utils/MemorySSA.cpp?rev=2934<wbr>53&r1=293452&r2=293453&view=<wbr>diff</a><br>
==============================<wbr>==============================<wbr>==================<br>
--- llvm/trunk/lib/Transforms/Util<wbr>s/MemorySSA.cpp (original)<br>
+++ llvm/trunk/lib/Transforms/Util<wbr>s/MemorySSA.cpp Sun Jan 29 21:16:43 2017<br>
@@ -1943,9 +1943,8 @@ void MemorySSA::verifyOrdering(Func<wbr>tion<br>
     assert(AL->size() == ActualAccesses.size() &&<br>
            "We don't have the same number of accesses in the block as on the "<br>
            "access list");<br>
-    assert(DL ||<br>
-           ActualDefs.size() == 0 &&<br>
-               "Either we should have a defs list, or we should have no defs");<br>
+    assert((DL || ActualDefs.size() == 0) &&<br>
+           "Either we should have a defs list, or we should have no defs");<br>
     assert((!DL || DL->size() == ActualDefs.size()) &&<br>
            "We don't have the same number of defs in the block as on the "<br>
            "def list");<br>
<br>
<br>
______________________________<wbr>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div></div>