diff options
| author | Tomas Matheson <tomas.matheson@arm.com> | 2021-06-18 10:57:59 +0100 |
|---|---|---|
| committer | Tomas Matheson <tomas.matheson@arm.com> | 2021-06-19 17:02:11 +0100 |
| commit | 1bcfa84ae908ddcf042163345ffee2d247669e60 (patch) | |
| tree | 71f2501457bc19ac2aae15334f70859fe083609d /llvm/lib/Analysis/LazyCallGraph.cpp | |
| parent | 18dbe6897857944653ee8f68dca8d63bc38aaff8 (diff) | |
| download | llvm-1bcfa84ae908ddcf042163345ffee2d247669e60.zip llvm-1bcfa84ae908ddcf042163345ffee2d247669e60.tar.gz llvm-1bcfa84ae908ddcf042163345ffee2d247669e60.tar.bz2 | |
Allow building for release with EXPENSIVE_CHECKS
D97225 moved LazyCallGraph verify() calls behind EXPENSIVE_CHECKS,
but verity() is defined for debug builds only so this had the unintended
effect of breaking release builds with EXPENSIVE_CHECKS.
Fix by enabling verify() for both debug and EXPENSIVE_CHECKS.
Differential Revision: https://reviews.llvm.org/D104514
Diffstat (limited to 'llvm/lib/Analysis/LazyCallGraph.cpp')
| -rw-r--r-- | llvm/lib/Analysis/LazyCallGraph.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/llvm/lib/Analysis/LazyCallGraph.cpp b/llvm/lib/Analysis/LazyCallGraph.cpp index 16ea59e..8f87552 100644 --- a/llvm/lib/Analysis/LazyCallGraph.cpp +++ b/llvm/lib/Analysis/LazyCallGraph.cpp @@ -241,7 +241,7 @@ LLVM_DUMP_METHOD void LazyCallGraph::SCC::dump() const { } #endif -#ifndef NDEBUG +#if !defined(NDEBUG) || defined(EXPENSIVE_CHECKS) void LazyCallGraph::SCC::verify() { assert(OuterRefSCC && "Can't have a null RefSCC!"); assert(!Nodes.empty() && "Can't have an empty SCC!"); @@ -333,7 +333,7 @@ LLVM_DUMP_METHOD void LazyCallGraph::RefSCC::dump() const { } #endif -#ifndef NDEBUG +#if !defined(NDEBUG) || defined(EXPENSIVE_CHECKS) void LazyCallGraph::RefSCC::verify() { assert(G && "Can't have a null graph!"); assert(!SCCs.empty() && "Can't have an empty SCC!"); |