diff options
author | Sanjoy Das <sanjoy@playingwithpointers.com> | 2015-02-23 23:13:22 +0000 |
---|---|---|
committer | Sanjoy Das <sanjoy@playingwithpointers.com> | 2015-02-23 23:13:22 +0000 |
commit | c9cf0151cf73ed44dd3af74568a9c01d0bded8b9 (patch) | |
tree | 7f8d5c477151ea5d19c68cc5f17423100cf531a6 /llvm/lib | |
parent | 662c1d2770c829222c9cb3f0c3bff167b561483f (diff) | |
download | bcm5719-llvm-c9cf0151cf73ed44dd3af74568a9c01d0bded8b9.tar.gz bcm5719-llvm-c9cf0151cf73ed44dd3af74568a9c01d0bded8b9.zip |
Revert 230275.
230275 got committed with an incorrect commit message due to a mixup
on my side. Will re-land in a few moments with the correct commit
message.
llvm-svn: 230279
Diffstat (limited to 'llvm/lib')
-rw-r--r-- | llvm/lib/Analysis/ScalarEvolutionExpander.cpp | 33 |
1 files changed, 2 insertions, 31 deletions
diff --git a/llvm/lib/Analysis/ScalarEvolutionExpander.cpp b/llvm/lib/Analysis/ScalarEvolutionExpander.cpp index ef9557132fc..59f19a002ec 100644 --- a/llvm/lib/Analysis/ScalarEvolutionExpander.cpp +++ b/llvm/lib/Analysis/ScalarEvolutionExpander.cpp @@ -1063,34 +1063,6 @@ static bool canBeCheaplyTransformed(ScalarEvolution &SE, return false; } -static bool IsIncrementNSW(ScalarEvolution &SE, const SCEVAddRecExpr *AR) { - if (!isa<IntegerType>(AR->getType())) - return false; - - unsigned BitWidth = cast<IntegerType>(AR->getType())->getBitWidth(); - Type *WideTy = IntegerType::get(AR->getType()->getContext(), BitWidth * 2); - const SCEV *Step = AR->getStepRecurrence(SE); - const SCEV *OpAfterExtend = SE.getAddExpr(SE.getSignExtendExpr(Step, WideTy), - SE.getSignExtendExpr(AR, WideTy)); - const SCEV *ExtendAfterOp = - SE.getSignExtendExpr(SE.getAddExpr(AR, Step), WideTy); - return ExtendAfterOp == OpAfterExtend; -} - -static bool IsIncrementNUW(ScalarEvolution &SE, const SCEVAddRecExpr *AR) { - if (!isa<IntegerType>(AR->getType())) - return false; - - unsigned BitWidth = cast<IntegerType>(AR->getType())->getBitWidth(); - Type *WideTy = IntegerType::get(AR->getType()->getContext(), BitWidth * 2); - const SCEV *Step = AR->getStepRecurrence(SE); - const SCEV *OpAfterExtend = SE.getAddExpr(SE.getZeroExtendExpr(Step, WideTy), - SE.getZeroExtendExpr(AR, WideTy)); - const SCEV *ExtendAfterOp = - SE.getZeroExtendExpr(SE.getAddExpr(AR, Step), WideTy); - return ExtendAfterOp == OpAfterExtend; -} - /// getAddRecExprPHILiterally - Helper for expandAddRecExprLiterally. Expand /// the base addrec, which is the addrec without any non-loop-dominating /// values, and return the PHI. @@ -1241,11 +1213,10 @@ SCEVExpander::getAddRecExprPHILiterally(const SCEVAddRecExpr *Normalized, IVIncInsertPos : Pred->getTerminator(); Builder.SetInsertPoint(InsertPos); Value *IncV = expandIVInc(PN, StepV, L, ExpandTy, IntTy, useSubtract); - if (isa<OverflowingBinaryOperator>(IncV)) { - if (IsIncrementNUW(SE, Normalized)) + if (Normalized->getNoWrapFlags(SCEV::FlagNUW)) cast<BinaryOperator>(IncV)->setHasNoUnsignedWrap(); - if (IsIncrementNSW(SE, Normalized)) + if (Normalized->getNoWrapFlags(SCEV::FlagNSW)) cast<BinaryOperator>(IncV)->setHasNoSignedWrap(); } PN->addIncoming(IncV, Pred); |