diff options
author | Davide Italiano <davide@freebsd.org> | 2017-04-26 21:28:40 +0000 |
---|---|---|
committer | Davide Italiano <davide@freebsd.org> | 2017-04-26 21:28:40 +0000 |
commit | d7b2a9981cf3b20147e41719cd9a00e0a56083ae (patch) | |
tree | 80600411f8a33b7a6b07cc1a732713a1f2435392 /llvm/lib/Transforms | |
parent | cf8247ef4850f4568dc54295a2e673b87065c436 (diff) | |
download | bcm5719-llvm-d7b2a9981cf3b20147e41719cd9a00e0a56083ae.tar.gz bcm5719-llvm-d7b2a9981cf3b20147e41719cd9a00e0a56083ae.zip |
[LibCallsShrinkWrap] Remove an unnecessary class member variable.
llvm-svn: 301477
Diffstat (limited to 'llvm/lib/Transforms')
-rw-r--r-- | llvm/lib/Transforms/Utils/LibCallsShrinkWrap.cpp | 15 |
1 files changed, 7 insertions, 8 deletions
diff --git a/llvm/lib/Transforms/Utils/LibCallsShrinkWrap.cpp b/llvm/lib/Transforms/Utils/LibCallsShrinkWrap.cpp index 5ce8785337c..42aca757c2a 100644 --- a/llvm/lib/Transforms/Utils/LibCallsShrinkWrap.cpp +++ b/llvm/lib/Transforms/Utils/LibCallsShrinkWrap.cpp @@ -74,10 +74,10 @@ namespace { class LibCallsShrinkWrap : public InstVisitor<LibCallsShrinkWrap> { public: LibCallsShrinkWrap(const TargetLibraryInfo &TLI, DominatorTree *DT) - : TLI(TLI), DT(DT), Changed(false){}; - bool isChanged() const { return Changed; } + : TLI(TLI), DT(DT){}; void visitCallInst(CallInst &CI) { checkCandidate(CI); } - void perform() { + bool perform() { + bool Changed = false; for (auto &CI : WorkList) { DEBUG(dbgs() << "CDCE calls: " << CI->getCalledFunction()->getName() << "\n"); @@ -86,6 +86,7 @@ public: DEBUG(dbgs() << "Transformed\n"); } } + return Changed; } private: @@ -128,7 +129,6 @@ private: const TargetLibraryInfo &TLI; DominatorTree *DT; SmallVector<CallInst *, 16> WorkList; - bool Changed; }; } // end anonymous namespace @@ -526,14 +526,14 @@ static bool runImpl(Function &F, const TargetLibraryInfo &TLI, return false; LibCallsShrinkWrap CCDCE(TLI, DT); CCDCE.visit(F); - CCDCE.perform(); + bool Changed = CCDCE.perform(); // Verify the dominator after we've updated it locally. #ifndef NDEBUG if (DT) DT->verifyDomTree(); #endif - return CCDCE.isChanged(); + return Changed; } bool LibCallsShrinkWrapLegacyPass::runOnFunction(Function &F) { @@ -555,8 +555,7 @@ PreservedAnalyses LibCallsShrinkWrapPass::run(Function &F, FunctionAnalysisManager &FAM) { auto &TLI = FAM.getResult<TargetLibraryAnalysis>(F); auto *DT = FAM.getCachedResult<DominatorTreeAnalysis>(F); - bool Changed = runImpl(F, TLI, DT); - if (!Changed) + if (!runImpl(F, TLI, DT)) return PreservedAnalyses::all(); auto PA = PreservedAnalyses(); PA.preserve<GlobalsAA>(); |