diff options
| author | Filipe Cabecinhas <me@filcab.net> | 2016-04-29 15:22:48 +0000 |
|---|---|---|
| committer | Filipe Cabecinhas <me@filcab.net> | 2016-04-29 15:22:48 +0000 |
| commit | 0da993751775f53c57bc9100b152626a8b380dfb (patch) | |
| tree | 00ebc346d265d804fb408ef1ff333691225ca188 /llvm/include | |
| parent | fee94ff8a996050547b0ba8cce2cda730b304b8f (diff) | |
| download | bcm5719-llvm-0da993751775f53c57bc9100b152626a8b380dfb.tar.gz bcm5719-llvm-0da993751775f53c57bc9100b152626a8b380dfb.zip | |
Unify XDEBUG and EXPENSIVE_CHECKS (into the latter), and add an option to the cmake build to enable them.
Summary:
Historically, we had a switch in the Makefiles for turning on "expensive
checks". This has never been ported to the cmake build, but the
(dead-ish) code is still around.
This will also make it easier to turn it on in buildbots.
Reviewers: chandlerc
Subscribers: jyknight, mzolotukhin, RKSimon, gberry, llvm-commits
Differential Revision: http://reviews.llvm.org/D19723
llvm-svn: 268050
Diffstat (limited to 'llvm/include')
| -rw-r--r-- | llvm/include/llvm/Analysis/RegionInfoImpl.h | 6 | ||||
| -rw-r--r-- | llvm/include/llvm/Support/GenericDomTree.h | 2 |
2 files changed, 4 insertions, 4 deletions
diff --git a/llvm/include/llvm/Analysis/RegionInfoImpl.h b/llvm/include/llvm/Analysis/RegionInfoImpl.h index e8c95e26585..15dd1a2000e 100644 --- a/llvm/include/llvm/Analysis/RegionInfoImpl.h +++ b/llvm/include/llvm/Analysis/RegionInfoImpl.h @@ -665,7 +665,7 @@ typename Tr::RegionT *RegionInfoBase<Tr>::createRegion(BlockT *entry, new RegionT(entry, exit, static_cast<RegionInfoT *>(this), DT); BBtoRegion.insert(std::make_pair(entry, region)); -#ifdef XDEBUG +#ifdef EXPENSIVE_CHECKS region->verifyRegion(); #else DEBUG(region->verifyRegion()); @@ -764,7 +764,7 @@ void RegionInfoBase<Tr>::buildRegionsTree(DomTreeNodeT *N, RegionT *region) { } } -#ifdef XDEBUG +#ifdef EXPENSIVE_CHECKS template <class Tr> bool RegionInfoBase<Tr>::VerifyRegionInfo = true; #else @@ -798,7 +798,7 @@ void RegionInfoBase<Tr>::releaseMemory() { template <class Tr> void RegionInfoBase<Tr>::verifyAnalysis() const { - // Do only verify regions if explicitely activated using XDEBUG or + // Do only verify regions if explicitely activated using EXPENSIVE_CHECKS or // -verify-region-info if (!RegionInfoBase<Tr>::VerifyRegionInfo) return; diff --git a/llvm/include/llvm/Support/GenericDomTree.h b/llvm/include/llvm/Support/GenericDomTree.h index 8bae582d18c..6b65ff8205e 100644 --- a/llvm/include/llvm/Support/GenericDomTree.h +++ b/llvm/include/llvm/Support/GenericDomTree.h @@ -453,7 +453,7 @@ public: // Compare the result of the tree walk and the dfs numbers, if expensive // checks are enabled. -#ifdef XDEBUG +#ifdef EXPENSIVE_CHECKS assert((!DFSInfoValid || (dominatedBySlowTreeWalk(A, B) == B->DominatedBy(A))) && "Tree walk disagrees with dfs numbers!"); |
