diff options
author | Sanjoy Das <sanjoy@playingwithpointers.com> | 2015-09-22 00:10:57 +0000 |
---|---|---|
committer | Sanjoy Das <sanjoy@playingwithpointers.com> | 2015-09-22 00:10:57 +0000 |
commit | 5d9a8cbb68e7e6849af61f44352d962c90ecf1cb (patch) | |
tree | b920ce54c9ffd7d82b76e2528f42531a92f6d1fe /llvm/lib/Analysis/ScalarEvolution.cpp | |
parent | 9d06ab6ded7ff156acf19076dbf834424390dfe3 (diff) | |
download | bcm5719-llvm-5d9a8cbb68e7e6849af61f44352d962c90ecf1cb.tar.gz bcm5719-llvm-5d9a8cbb68e7e6849af61f44352d962c90ecf1cb.zip |
[SCEV] Use SaveAndRestore<T> instead of a hand rolled struct; NFCI.
`ClearWalkingBEDominatingCondsOnExit` is exactly `SaveAndRestore<bool>`,
so use `SaveAndRestore<bool>` instead.
llvm-svn: 248227
Diffstat (limited to 'llvm/lib/Analysis/ScalarEvolution.cpp')
-rw-r--r-- | llvm/lib/Analysis/ScalarEvolution.cpp | 15 |
1 files changed, 2 insertions, 13 deletions
diff --git a/llvm/lib/Analysis/ScalarEvolution.cpp b/llvm/lib/Analysis/ScalarEvolution.cpp index 262280c8a9c..df1e3eb183a 100644 --- a/llvm/lib/Analysis/ScalarEvolution.cpp +++ b/llvm/lib/Analysis/ScalarEvolution.cpp @@ -88,6 +88,7 @@ #include "llvm/Support/ErrorHandling.h" #include "llvm/Support/MathExtras.h" #include "llvm/Support/raw_ostream.h" +#include "llvm/Support/SaveAndRestore.h" #include <algorithm> using namespace llvm; @@ -6987,24 +6988,12 @@ ScalarEvolution::isLoopBackedgeGuardedByCond(const Loop *L, LoopContinuePredicate->getSuccessor(0) != L->getHeader())) return true; - struct ClearWalkingBEDominatingCondsOnExit { - ScalarEvolution &SE; - - explicit ClearWalkingBEDominatingCondsOnExit(ScalarEvolution &SE) - : SE(SE){} - - ~ClearWalkingBEDominatingCondsOnExit() { - SE.WalkingBEDominatingConds = false; - } - }; - // We don't want more than one activation of the following loops on the stack // -- that can lead to O(n!) time complexity. if (WalkingBEDominatingConds) return false; - WalkingBEDominatingConds = true; - ClearWalkingBEDominatingCondsOnExit ClearOnExit(*this); + SaveAndRestore<bool> ClearOnExit(WalkingBEDominatingConds, true); // Check conditions due to any @llvm.assume intrinsics. for (auto &AssumeVH : AC.assumptions()) { |