Two problems with these lines of code:

1. The "work" was not in the assert, so it was punishing the optimized release
 2. getNamedFunction is _very_ expensive in large programs.  It is not designed
    to be used like this, and was taking 7% of the execution time of the code
    generator on perlbmk.

Since the assert "can never fail", I'm just killing it.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@11214 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Chris Lattner 2004-02-09 00:59:07 +00:00
parent 99d6b8ec95
commit e21f68d1e8

View File

@ -91,8 +91,6 @@ FunctionPassManager::~FunctionPassManager() { delete PM; }
void FunctionPassManager::add(FunctionPass *P) { PM->add(P); }
void FunctionPassManager::add(ImmutablePass *IP) { PM->add(IP); }
bool FunctionPassManager::run(Function &F) {
Function *mF = MP->getModule()->getNamedFunction(F.getName());
assert((&F == mF) && "ModuleProvider does not contain this function!");
MP->materializeFunction(&F);
return PM->run(F);
}