[llvm-commits] [llvm] r51845 - /llvm/trunk/lib/Analysis/MemoryDependenceAnalysis.cpp

[Chris Lattner]

On Jun 1, 2008, at 1:51 PM, Owen Anderson wrote:

> Author: resistor
> Date: Sun Jun  1 15:51:41 2008
> New Revision: 51845
>
> URL: http://llvm.org/viewvc/llvm-project?rev=51845&view=rev
> Log:
> Make ping more aggressive in finding nonlocal caching errors.

Hey Owen,

Please wrap this loop with #ifndef NDEBUG, so that the iteration over  
the map isn't kept in no-assert mode.

-Chris

>
>
> Modified:
>    llvm/trunk/lib/Analysis/MemoryDependenceAnalysis.cpp
>
> Modified: llvm/trunk/lib/Analysis/MemoryDependenceAnalysis.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/MemoryDependenceAnalysis.cpp?rev=51845&r1=51844&r2=51845&view=diff
>
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> ======================================================================
> --- llvm/trunk/lib/Analysis/MemoryDependenceAnalysis.cpp (original)
> +++ llvm/trunk/lib/Analysis/MemoryDependenceAnalysis.cpp Sun Jun  1  
> 15:51:41 2008
> @@ -58,6 +58,9 @@
>   for (nonLocalDepMapType::iterator I = depGraphNonLocal.begin(), E  
> = depGraphNonLocal.end();
>        I != E; ++I) {
>     assert(I->first != D);
> +    for (DenseMap<BasicBlock*, Value*>::iterator II = I- 
> >second.begin(),
> +         EE = I->second.end(); II  != EE; ++II)
> +      assert(II->second != D);
>   }
>
>   for (reverseDepMapType::iterator I = reverseDep.begin(), E =  
> reverseDep.end();
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits