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

George Burgess IV via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 29 22:48:55 PST 2017


Probably. My mistake. :)

On Sun, Jan 29, 2017 at 10:44 PM, Daniel Berlin <dberlin at dberlin.org> wrote:

> 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/Transform
>> s/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/62922616/attachment.html>


More information about the llvm-commits mailing list