diff options
author | Nicolai Hähnle <nicolai.haehnle@amd.com> | 2020-05-18 16:28:26 +0200 |
---|---|---|
committer | Nicolai Hähnle <nicolai.haehnle@amd.com> | 2020-07-06 21:58:11 +0200 |
commit | 723a44c9b5d654ec791720fc450757ae00f9e631 (patch) | |
tree | f4f749f396f860f3180c98b94aa166698208e64a /llvm | |
parent | 76c5cb05a3a67340cc7950eb8fb5c2d2a0ac4554 (diff) | |
download | llvm-723a44c9b5d654ec791720fc450757ae00f9e631.zip llvm-723a44c9b5d654ec791720fc450757ae00f9e631.tar.gz llvm-723a44c9b5d654ec791720fc450757ae00f9e631.tar.bz2 |
DomTree: Remove the releaseMemory() method
Summary:
It is fully redundant with reset().
Change-Id: I25850b9f08eace757cf03cbb8780e970aca7f51a
Reviewers: arsenm, RKSimon, mehdi_amini, courbet
Subscribers: wdng, cfe-commits, llvm-commits
Tags: #clang, #llvm
Differential Revision: https://reviews.llvm.org/D83084
Diffstat (limited to 'llvm')
-rw-r--r-- | llvm/include/llvm/Analysis/PostDominators.h | 4 | ||||
-rw-r--r-- | llvm/include/llvm/IR/Dominators.h | 2 | ||||
-rw-r--r-- | llvm/include/llvm/Support/GenericDomTree.h | 8 |
3 files changed, 5 insertions, 9 deletions
diff --git a/llvm/include/llvm/Analysis/PostDominators.h b/llvm/include/llvm/Analysis/PostDominators.h index 801eb3d..296110d 100644 --- a/llvm/include/llvm/Analysis/PostDominators.h +++ b/llvm/include/llvm/Analysis/PostDominators.h @@ -88,9 +88,7 @@ struct PostDominatorTreeWrapperPass : public FunctionPass { AU.setPreservesAll(); } - void releaseMemory() override { - DT.releaseMemory(); - } + void releaseMemory() override { DT.reset(); } void print(raw_ostream &OS, const Module*) const override; }; diff --git a/llvm/include/llvm/IR/Dominators.h b/llvm/include/llvm/IR/Dominators.h index a79be87..0084ac0 100644 --- a/llvm/include/llvm/IR/Dominators.h +++ b/llvm/include/llvm/IR/Dominators.h @@ -277,7 +277,7 @@ public: AU.setPreservesAll(); } - void releaseMemory() override { DT.releaseMemory(); } + void releaseMemory() override { DT.reset(); } void print(raw_ostream &OS, const Module *M = nullptr) const override; }; diff --git a/llvm/include/llvm/Support/GenericDomTree.h b/llvm/include/llvm/Support/GenericDomTree.h index d932938..e83e7aa 100644 --- a/llvm/include/llvm/Support/GenericDomTree.h +++ b/llvm/include/llvm/Support/GenericDomTree.h @@ -325,8 +325,6 @@ protected: return false; } - void releaseMemory() { reset(); } - /// getNode - return the (Post)DominatorTree node for the specified basic /// block. This is the same as using operator[] on this class. The result /// may (but is not required to) be null for a forward (backwards) @@ -760,9 +758,6 @@ public: return DomTreeBuilder::Verify(*this, VL); } -protected: - void addRoot(NodeT *BB) { this->Roots.push_back(BB); } - void reset() { DomTreeNodes.clear(); Roots.clear(); @@ -772,6 +767,9 @@ protected: SlowQueries = 0; } +protected: + void addRoot(NodeT *BB) { this->Roots.push_back(BB); } + // NewBB is split and now it has one successor. Update dominator tree to // reflect this change. template <class N> |