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

Douglas Gregor dgregor at apple.com
Wed Sep 21 07:11:55 PDT 2011


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




More information about the cfe-commits mailing list