[llvm-commits] [llvm] r116785 - /llvm/trunk/lib/System/Win32/ThreadLocal.inc

[Jim Grosbach]

Can you add a comment saying why that line is necessary? Otherwise some well meaning person may remove the "unusued" statement.

-Jim

On Oct 18, 2010, at 6:22 PM, NAKAMURA Takumi wrote:

> Author: chapuni
> Date: Mon Oct 18 20:22:01 2010
> New Revision: 116785
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=116785&view=rev
> Log:
> lib/System/Win32/ThreadLocal.inc: Suppress "unused" warning on -Asserts.
> 
> Modified:
>    llvm/trunk/lib/System/Win32/ThreadLocal.inc
> 
> Modified: llvm/trunk/lib/System/Win32/ThreadLocal.inc
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/System/Win32/ThreadLocal.inc?rev=116785&r1=116784&r2=116785&view=diff
> ==============================================================================
> --- llvm/trunk/lib/System/Win32/ThreadLocal.inc (original)
> +++ llvm/trunk/lib/System/Win32/ThreadLocal.inc Mon Oct 18 20:22:01 2010
> @@ -44,6 +44,7 @@
>   DWORD* tls = static_cast<DWORD*>(data);
>   int errorcode = TlsSetValue(*tls, const_cast<void*>(d));
>   assert(errorcode != 0);
> +  (void)errorcode;
> }
> 
> void ThreadLocalImpl::removeInstance() {
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits