diff options
| author | Nuno Lopes <nunoplopes@sapo.pt> | 2008-11-04 23:03:58 +0000 |
|---|---|---|
| committer | Nuno Lopes <nunoplopes@sapo.pt> | 2008-11-04 23:03:58 +0000 |
| commit | 641397f2ef0a155159e1085bf594c1535485e3a3 (patch) | |
| tree | 155e99e8e66a3c25af0577bb5ad1a59c89aaf51a /lib/VMCore/PassManager.cpp | |
| parent | 879b1ac22e6307e3111eac87bfba123fba1ac7a4 (diff) | |
| download | external_llvm-641397f2ef0a155159e1085bf594c1535485e3a3.tar.gz external_llvm-641397f2ef0a155159e1085bf594c1535485e3a3.tar.bz2 external_llvm-641397f2ef0a155159e1085bf594c1535485e3a3.zip | |
fix memory leak in pass manager when adding an analysis pass that already existed. as pass manager takes ownership of the added passes, it has to delete the pass if it isnt added to the pass list
tweak the opt tool so that it doesnt access a Pass after the ownership was taken by the pass manager
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@58730 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/VMCore/PassManager.cpp')
| -rw-r--r-- | lib/VMCore/PassManager.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/lib/VMCore/PassManager.cpp b/lib/VMCore/PassManager.cpp index ef90aace3d..6f559c47c3 100644 --- a/lib/VMCore/PassManager.cpp +++ b/lib/VMCore/PassManager.cpp @@ -458,8 +458,10 @@ void PMTopLevelManager::schedulePass(Pass *P) { // generate the analysis again. Stale analysis info should not be // available at this point. if (P->getPassInfo() && - P->getPassInfo()->isAnalysis() && findAnalysisPass(P->getPassInfo())) + P->getPassInfo()->isAnalysis() && findAnalysisPass(P->getPassInfo())) { + delete P; return; + } AnalysisUsage *AnUsage = findAnalysisUsage(P); |