[llvm-commits] [llvm] r91821 - /llvm/trunk/lib/Transforms/Utils/LoopSimplify.cpp

[Evan Cheng]

I think 90260 is the test case.

Evan

On Dec 20, 2009, at 11:45 PM, Chris Lattner wrote:

> Author: lattner
> Date: Mon Dec 21 01:45:57 2009
> New Revision: 91821
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=91821&view=rev
> Log:
> revert r89298, which was committed without a testcase.  I think
> the underlying PHI node insertion issue in SSAUpdate is fixed.
> 
> Modified:
>    llvm/trunk/lib/Transforms/Utils/LoopSimplify.cpp
> 
> Modified: llvm/trunk/lib/Transforms/Utils/LoopSimplify.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Utils/LoopSimplify.cpp?rev=91821&r1=91820&r2=91821&view=diff
> 
> ==============================================================================
> --- llvm/trunk/lib/Transforms/Utils/LoopSimplify.cpp (original)
> +++ llvm/trunk/lib/Transforms/Utils/LoopSimplify.cpp Mon Dec 21 01:45:57 2009
> @@ -305,12 +305,6 @@
>     }
>   }
> 
> -  // If there are duplicate phi nodes (for example, from loop rotation),
> -  // get rid of them.
> -  for (Loop::block_iterator BB = L->block_begin(), E = L->block_end();
> -       BB != E; ++BB)
> -    EliminateDuplicatePHINodes(*BB);
> -
>   return Changed;
> }
> 
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits