[cfe-commits] r140232 - /cfe/trunk/lib/Sema/SemaExpr.cpp

Richard Trieu rtrieu at google.com
Wed Sep 21 12:31:20 PDT 2011


On Wed, Sep 21, 2011 at 7:11 AM, Douglas Gregor <dgregor at apple.com> wrote:
>
> On Sep 20, 2011, at 7:50 PM, Richard Trieu wrote:
>
>> Author: rtrieu
>> Date: Tue Sep 20 21:50:14 2011
>> New Revision: 140232
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=140232&view=rev
>> Log:
>> Change:
>>
>>  assert(!"error message");
>>
>> To:
>>
>>  assert(0 && "error message");
>>
>> which is more consistant across the code base.
>>
>> Modified:
>>    cfe/trunk/lib/Sema/SemaExpr.cpp
>>
>> Modified: cfe/trunk/lib/Sema/SemaExpr.cpp
>> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaExpr.cpp?rev=140232&r1=140231&r2=140232&view=diff
>> ==============================================================================
>> --- cfe/trunk/lib/Sema/SemaExpr.cpp (original)
>> +++ cfe/trunk/lib/Sema/SemaExpr.cpp Tue Sep 20 21:50:14 2011
>> @@ -8936,7 +8936,7 @@
>>   else if (pw == Context.getTargetInfo().getLongLongWidth())
>>     Ty = Context.LongLongTy;
>>   else {
>> -    assert(!"I don't know size of pointer!");
>> +    assert(0 && "I don't know size of pointer!");
>>     Ty = Context.IntTy;
>
> Just use llvm_unreachable?
>
>        - Doug
>
>

Should all assert(0 && "error") be switched to llvm_unreachable("error") ?




More information about the cfe-commits mailing list