'Pass' should now not be derived from by clients. Instead, they should derive

from ModulePass.  Instead of implementing Pass::run, then should implement
ModulePass::runOnModule.

Also, fix some undefined behavior, expecting | on booleans to evaluate
left-to-right.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@16435 91177308-0d34-0410-b5e6-96231b3b80d8

1 files changed, 19 insertions, 10 deletions

diff --git a/lib/VMCore/Pass.cpp b/lib/VMCore/Pass.cpp
index 5079b4ea65..87748d83b4 100644
--- a/lib/VMCore/Pass.cpp
+++ b/lib/VMCore/Pass.cpp
@@ -78,8 +78,12 @@ void AnalysisUsage::setPreservesCFG() {
 //
 PassManager::PassManager() : PM(new PassManagerT<Module>()) {}
 PassManager::~PassManager() { delete PM; }
-void PassManager::add(Pass *P) { PM->add(P); }
-bool PassManager::run(Module &M) { return PM->run(M); }
+void PassManager::add(Pass *P) {
+  ModulePass *MP = dynamic_cast<ModulePass*>(P);
+  assert(MP && "Not a modulepass?");
+  PM->add(MP);
+}
+bool PassManager::run(Module &M) { return PM->runOnModule(M); }
 
 //===----------------------------------------------------------------------===//
 // FunctionPassManager implementation - The FunctionPassManager class
@@ -101,7 +105,7 @@ bool FunctionPassManager::run(Function &F) {
     std::cerr << "Error reading bytecode file!\n";
     abort();
   }
-  return PM->run(F); 
+  return PM->run(F);
 }
 
 
@@ -188,7 +192,7 @@ void PMDebug::PrintAnalysisSetInfo(unsigned Depth, const char *Msg,
 // Pass Implementation
 //
 
-void Pass::addToPassManager(PassManagerT<Module> *PM, AnalysisUsage &AU) {
+void ModulePass::addToPassManager(PassManagerT<Module> *PM, AnalysisUsage &AU) {
   PM->addPass(this, AU);
 }
 
@@ -238,7 +242,7 @@ void ImmutablePass::addToPassManager(PassManagerT<Module> *PM,
 // run - On a module, we run this pass by initializing, runOnFunction'ing once
 // for every function in the module, then by finalizing.
 //
-bool FunctionPass::run(Module &M) {
+bool FunctionPass::runOnModule(Module &M) {
   bool Changed = doInitialization(M);
   
   for (Module::iterator I = M.begin(), E = M.end(); I != E; ++I)
@@ -253,8 +257,9 @@ bool FunctionPass::run(Module &M) {
 bool FunctionPass::run(Function &F) {
   if (F.isExternal()) return false;// Passes are not run on external functions!
 
-  return doInitialization(*F.getParent()) | runOnFunction(F)
-       | doFinalization(*F.getParent());
+  bool Changed = doInitialization(*F.getParent());
+  Changed |= runOnFunction(F);
+  return Changed | doFinalization(*F.getParent());
 }
 
 void FunctionPass::addToPassManager(PassManagerT<Module> *PM,
@@ -284,11 +289,15 @@ bool BasicBlockPass::runOnFunction(Function &F) {
 // To run directly on the basic block, we initialize, runOnBasicBlock, then
 // finalize.
 //
-bool BasicBlockPass::run(BasicBlock &BB) {
+bool BasicBlockPass::runPass(BasicBlock &BB) {
   Function &F = *BB.getParent();
   Module &M = *F.getParent();
-  return doInitialization(M) | doInitialization(F) | runOnBasicBlock(BB) |
-         doFinalization(F) | doFinalization(M);
+  bool Changed = doInitialization(M);
+  Changed |= doInitialization(F);
+  Changed |= runOnBasicBlock(BB);
+  Changed |= doFinalization(F);
+  Changed |= doFinalization(M);
+  return Changed;
 }
 
 void BasicBlockPass::addToPassManager(PassManagerT<Function> *PM,