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

[Owen Anderson]

Yup.  Sorry, will fix when I get back to my development machine.

--Owen

 
On Monday, April 07, 2008, at 12:49PM, "Török Edwin" <edwintorok at gmail.com> wrote:
>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
>_______________________________________________
>llvm-commits mailing list
>llvm-commits at cs.uiuc.edu
>http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>