On Wed, Sep 21, 2011 at 7:11 AM, Douglas Gregor <[email protected]> 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") ?
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
