[cfe-commits] r78305 - /cfe/trunk/lib/Analysis/RegionStore.cpp

Daniel Dunbar daniel at zuster.org
Thu Aug 6 08:27:13 PDT 2009


On Thu, Aug 6, 2009 at 5:56 AM, Mike Stump<mrs at apple.com> wrote:
> Author: mrs
> Date: Thu Aug  6 07:56:50 2009
> New Revision: 78305
>
> URL: http://llvm.org/viewvc/llvm-project?rev=78305&view=rev
> Log:
> I have a dream, one day, we won't need to do this.

Is anything blocking this dream? assert() could be defined to simply
if(0) {} out the code when in NDEBUG.

 - Daniel

>
> Modified:
>    cfe/trunk/lib/Analysis/RegionStore.cpp
>
> Modified: cfe/trunk/lib/Analysis/RegionStore.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Analysis/RegionStore.cpp?rev=78305&r1=78304&r2=78305&view=diff
>
> ==============================================================================
> --- cfe/trunk/lib/Analysis/RegionStore.cpp (original)
> +++ cfe/trunk/lib/Analysis/RegionStore.cpp Thu Aug  6 07:56:50 2009
> @@ -1230,6 +1230,7 @@
>   const RecordType* RT = T->getAsStructureType();
>   RecordDecl* RD = RT->getDecl();
>   assert(RD->isDefinition());
> +  (void)RD;
>  #if USE_EXPLICIT_COMPOUND
>   llvm::ImmutableList<SVal> StructVal = getBasicVals().getEmptySValList();
>
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>




More information about the cfe-commits mailing list