[llvm-commits] [llvm] r107496 - /llvm/trunk/lib/Transforms/Scalar/DeadStoreElimination.cpp

[Chris Lattner]

On Jul 2, 2010, at 11:43 AM, Dan Gohman wrote:

> Author: djg
> Date: Fri Jul  2 13:43:05 2010
> New Revision: 107496
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=107496&view=rev
> Log:
> Don't claim to preserve AliasAnalysis. First, this is doesn't actually
> have any effect, and second, deleting stores can potentially invalidate
> an AliasAnalysis, and there's currently no notification for this.

Alias analysis is preserved.  Through statements like this:
  MDA.removeInstruction(DeadInst);

Memdep updates AA.

-Chris


> 
> Modified:
>    llvm/trunk/lib/Transforms/Scalar/DeadStoreElimination.cpp
> 
> Modified: llvm/trunk/lib/Transforms/Scalar/DeadStoreElimination.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/DeadStoreElimination.cpp?rev=107496&r1=107495&r2=107496&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Transforms/Scalar/DeadStoreElimination.cpp (original)
> +++ llvm/trunk/lib/Transforms/Scalar/DeadStoreElimination.cpp Fri Jul  2 13:43:05 2010
> @@ -74,7 +74,6 @@
>       AU.addRequired<AliasAnalysis>();
>       AU.addRequired<MemoryDependenceAnalysis>();
>       AU.addPreserved<DominatorTree>();
> -      AU.addPreserved<AliasAnalysis>();
>       AU.addPreserved<MemoryDependenceAnalysis>();
>     }
> 
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits