[llvm] r293453 - [MemorySSA] Correct an assertion surrounding with parentheses.

[Daniel Berlin via llvm-commits]

I think this was one george asked me to remove parens from in the review?


On Sun, Jan 29, 2017 at 7:16 PM, Davide Italiano via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> Author: davide
> Date: Sun Jan 29 21:16:43 2017
> New Revision: 293453
>
> URL: http://llvm.org/viewvc/llvm-project?rev=293453&view=rev
> Log:
> [MemorySSA] Correct an assertion surrounding with parentheses.
>
> Modified:
>     llvm/trunk/lib/Transforms/Utils/MemorySSA.cpp
>
> Modified: llvm/trunk/lib/Transforms/Utils/MemorySSA.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/
> Transforms/Utils/MemorySSA.cpp?rev=293453&r1=293452&r2=293453&view=diff
> ============================================================
> ==================
> --- llvm/trunk/lib/Transforms/Utils/MemorySSA.cpp (original)
> +++ llvm/trunk/lib/Transforms/Utils/MemorySSA.cpp Sun Jan 29 21:16:43 2017
> @@ -1943,9 +1943,8 @@ void MemorySSA::verifyOrdering(Function
>      assert(AL->size() == ActualAccesses.size() &&
>             "We don't have the same number of accesses in the block as on
> the "
>             "access list");
> -    assert(DL ||
> -           ActualDefs.size() == 0 &&
> -               "Either we should have a defs list, or we should have no
> defs");
> +    assert((DL || ActualDefs.size() == 0) &&
> +           "Either we should have a defs list, or we should have no
> defs");
>      assert((!DL || DL->size() == ActualDefs.size()) &&
>             "We don't have the same number of defs in the block as on the "
>             "def list");
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170129/834b169d/attachment.html>