commit d7b2a9981cf3b20147e41719cd9a00e0a56083ae
author Davide Italiano <davide@freebsd.org> Wed Apr 26 21:28:40 2017 +0000
committer Davide Italiano <davide@freebsd.org> Wed Apr 26 21:28:40 2017 +0000
tree 80600411f8a33b7a6b07cc1a732713a1f2435392
parent cf8247ef4850f4568dc54295a2e673b87065c436

[LibCallsShrinkWrap] Remove an unnecessary class member variable.

llvm-svn: 301477

llvm/lib/Transforms/Utils/LibCallsShrinkWrap.cpp

diff --git a/llvm/lib/Transforms/Utils/LibCallsShrinkWrap.cpp b/llvm/lib/Transforms/Utils/LibCallsShrinkWrap.cpp
index 5ce8785..42aca75 100644
--- a/llvm/lib/Transforms/Utils/LibCallsShrinkWrap.cpp
+++ b/llvm/lib/Transforms/Utils/LibCallsShrinkWrap.cpp
@@ -74,10 +74,10 @@
 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 @@
         DEBUG(dbgs() << "Transformed\n");
       }
     }
+    return Changed;
   }
 
 private:
@@ -128,7 +129,6 @@
   const TargetLibraryInfo &TLI;
   DominatorTree *DT;
   SmallVector<CallInst *, 16> WorkList;
-  bool Changed;
 };
 } // end anonymous namespace
 
@@ -526,14 +526,14 @@
     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 @@
                                               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>();