[llvm-commits] CVS: llvm/lib/System/DynamicLibrary.cpp

Reid Spencer rspencer at reidspencer.com
Wed Aug 30 13:52:09 PDT 2006


Oops!

On Wed, 2006-08-30 at 15:37 -0500, Chris Lattner wrote:
> 
> Changes in directory llvm/lib/System:
> 
> DynamicLibrary.cpp updated: 1.17 -> 1.18
> ---
> Log message:
> 
> Guess what happens when asserts are disabled.  :(
> 
> Also, the assert could never fire due to || instead of &&.
> 
> 
> ---
> Diffs of the changes:  (+2 -1)
> 
>  DynamicLibrary.cpp |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletion(-)
> 
> 
> Index: llvm/lib/System/DynamicLibrary.cpp
> diff -u llvm/lib/System/DynamicLibrary.cpp:1.17 llvm/lib/System/DynamicLibrary.cpp:1.18
> --- llvm/lib/System/DynamicLibrary.cpp:1.17	Fri Aug 25 14:54:53 2006
> +++ llvm/lib/System/DynamicLibrary.cpp	Wed Aug 30 15:37:06 2006
> @@ -48,7 +48,8 @@
>  static inline void check_ltdl_initialization() {
>    static bool did_initialize_ltdl = false;
>    if (!did_initialize_ltdl) {
> -    assert(0 == lt_dlinit() || "Can't init the ltdl library");
> +    int Err = lt_dlinit();
> +    assert(0 == Err && "Can't init the ltdl library");
>      did_initialize_ltdl = true;
>    }
>  }
> 
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits




More information about the llvm-commits mailing list