[llvm-commits] [llvm] r49345 - in /llvm/trunk: include/llvm/ADT/SparseBitVector.h lib/Transforms/Scalar/GVN.cpp

[Török Edwin]

Owen Anderson wrote:
> @@ -1598,6 +1586,10 @@
>    if (isa<AllocationInst>(I))
>      return false;
>    
> +  // Allocations are always unique, so don't bother value numbering them.
> +  if (isa<AllocationInst>(I))
> +    return false;
> +  
>    if (MemCpyInst* M = dyn_cast<MemCpyInst>(I)) {
>      MemoryDependenceAnalysis& MD = getAnalysis<MemoryDependenceAnalysis>();
>   

Why do same test twice? Patch applied  twice?

Best regards,
--Edwin