This is an archive of the discontinued LLVM Phabricator instance.

[Dominators] Update Clang's DominatorTree to use the new template argument
ClosedPublic

Authored by kuhar on Jul 12 2017, 11:06 AM.

Reviewers

Commits

Summary

This patch makes the Clang's DominatorTree use the new IsPostDom template argument for DominatorTreeBase.

This revision is now accepted and ready to land.Jul 13 2017, 10:33 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

cfe/

trunk/

include/

clang/

Analysis/

Analyses/

6 lines

	Show All 32 Lines
	typedef llvm::DomTreeNodeBase<CFGBlock> DomTreeNode;			typedef llvm::DomTreeNodeBase<CFGBlock> DomTreeNode;

	/// \brief Concrete subclass of DominatorTreeBase for Clang			/// \brief Concrete subclass of DominatorTreeBase for Clang
	/// This class implements the dominators tree functionality given a Clang CFG.			/// This class implements the dominators tree functionality given a Clang CFG.
	///			///
	class DominatorTree : public ManagedAnalysis {			class DominatorTree : public ManagedAnalysis {
	virtual void anchor();			virtual void anchor();
	public:			public:
	llvm::DominatorTreeBase<CFGBlock>* DT;			llvm::DomTreeBase<CFGBlock>* DT;

	DominatorTree() {			DominatorTree() {
	DT = new llvm::DominatorTreeBase<CFGBlock>(false);			DT = new llvm::DomTreeBase<CFGBlock>();
	}			}

	~DominatorTree() override { delete DT; }			~DominatorTree() override { delete DT; }

	llvm::DominatorTreeBase<CFGBlock>& getBase() { return *DT; }			llvm::DomTreeBase<CFGBlock>& getBase() { return *DT; }

	/// \brief This method returns the root CFGBlock of the dominators tree.			/// \brief This method returns the root CFGBlock of the dominators tree.
	///			///
	inline CFGBlock *getRoot() const {			inline CFGBlock *getRoot() const {
	return DT->getRoot();			return DT->getRoot();
	}			}

	/// \brief This method returns the root DomTreeNode, which is the wrapper			/// \brief This method returns the root DomTreeNode, which is the wrapper
	▲ Show 20 Lines • Show All 148 Lines • Show Last 20 Lines