diff options
author | Andrew Trick <atrick@apple.com> | 2015-05-18 16:49:31 +0000 |
---|---|---|
committer | Andrew Trick <atrick@apple.com> | 2015-05-18 16:49:31 +0000 |
commit | 018e55a18751000a153d4bd3ed4637970fb6f03e (patch) | |
tree | 4d090ef0b474cad29396a87b72ceae6d357ee40e /llvm | |
parent | c49e78851cbb665bce954f68ac202ef25626d5a1 (diff) | |
download | bcm5719-llvm-018e55a18751000a153d4bd3ed4637970fb6f03e.tar.gz bcm5719-llvm-018e55a18751000a153d4bd3ed4637970fb6f03e.zip |
SimplifyIV comments and dead argument cleanup.
Remove crufty comments. IVUsers hasn't been used here for a long time.
llvm-svn: 237586
Diffstat (limited to 'llvm')
-rw-r--r-- | llvm/lib/Transforms/Scalar/IndVarSimplify.cpp | 9 | ||||
-rw-r--r-- | llvm/lib/Transforms/Utils/SimplifyIndVar.cpp | 3 |
2 files changed, 5 insertions, 7 deletions
diff --git a/llvm/lib/Transforms/Scalar/IndVarSimplify.cpp b/llvm/lib/Transforms/Scalar/IndVarSimplify.cpp index ab8e5b8cc02..2ba3109354b 100644 --- a/llvm/lib/Transforms/Scalar/IndVarSimplify.cpp +++ b/llvm/lib/Transforms/Scalar/IndVarSimplify.cpp @@ -622,9 +622,8 @@ void IndVarSimplify::RewriteLoopExitValues(Loop *L, SCEVExpander &Rewriter) { } } - // If we were unable to completely replace the PHI node, clone the PHI - // and delete the original one. This lets IVUsers and any other maps - // purge the original user from their records. + // If we were unable to completely replace the PHI node, clone the PHI and + // delete the original one. This purges the original phi. if (!LCSSASafePhiForRAUW) { PHINode *NewPN = cast<PHINode>(PN->clone()); NewPN->takeName(PN); @@ -915,8 +914,8 @@ const SCEVAddRecExpr* WidenIV::GetExtendedOperandRecurrence(NarrowIVDefUse DU) { return AddRec; } -/// GetWideRecurrence - Is this instruction potentially interesting from -/// IVUsers' perspective after widening it's type? In other words, can the +/// GetWideRecurrence - Is this instruction potentially interesting for further +/// simplification after widening it's type? In other words, can the /// extend be safely hoisted out of the loop with SCEV reducing the value to a /// recurrence on the same loop. If so, return the sign or zero extended /// recurrence. Otherwise return NULL. diff --git a/llvm/lib/Transforms/Utils/SimplifyIndVar.cpp b/llvm/lib/Transforms/Utils/SimplifyIndVar.cpp index 8bfc5fbca6f..3757a801d64 100644 --- a/llvm/lib/Transforms/Utils/SimplifyIndVar.cpp +++ b/llvm/lib/Transforms/Utils/SimplifyIndVar.cpp @@ -17,7 +17,6 @@ #include "llvm/ADT/STLExtras.h" #include "llvm/ADT/SmallVector.h" #include "llvm/ADT/Statistic.h" -#include "llvm/Analysis/IVUsers.h" #include "llvm/Analysis/LoopInfo.h" #include "llvm/Analysis/LoopPass.h" #include "llvm/Analysis/ScalarEvolutionExpressions.h" @@ -55,7 +54,7 @@ namespace { public: SimplifyIndvar(Loop *Loop, ScalarEvolution *SE, LoopInfo *LI, - SmallVectorImpl<WeakVH> &Dead, IVUsers *IVU = nullptr) + SmallVectorImpl<WeakVH> &Dead) : L(Loop), LI(LI), SE(SE), DeadInsts(Dead), Changed(false) { assert(LI && "IV simplification requires LoopInfo"); } |