Avoid assigning to Changed when it won't be used after the return.
authorBill Wendling <isanbard@gmail.com>
Fri, 25 Dec 2009 13:50:18 +0000 (13:50 +0000)
committerBill Wendling <isanbard@gmail.com>
Fri, 25 Dec 2009 13:50:18 +0000 (13:50 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@92160 91177308-0d34-0410-b5e6-96231b3b80d8

lib/VMCore/PassManager.cpp

index 52e8a8269a023923367eb5cad7a27ffcf197c202..d68838584029e697e7d90638698674122b4d5b01 100644 (file)
@@ -1133,7 +1133,7 @@ bool BBPassManager::runOnFunction(Function &F) {
       removeDeadPasses(BP, I->getName(), ON_BASICBLOCK_MSG);
     }
 
-  return Changed |= doFinalization(F);
+  return doFinalization(F) || Changed;
 }
 
 // Implement doInitialization and doFinalization
@@ -1355,7 +1355,7 @@ bool FPPassManager::runOnModule(Module &M) {
   for (Module::iterator I = M.begin(), E = M.end(); I != E; ++I)
     runOnFunction(*I);
 
-  return Changed |= doFinalization(M);
+  return doFinalization(M) || Changed;
 }
 
 bool FPPassManager::doInitialization(Module &M) {