diff options
| author | James Y Knight <jyknight@google.com> | 2016-08-30 03:16:16 +0000 |
|---|---|---|
| committer | James Y Knight <jyknight@google.com> | 2016-08-30 03:16:16 +0000 |
| commit | d7d9e1069be675f40dd89e27143c0dc176ed498d (patch) | |
| tree | e9841820a70e198799f8dd839bd6a3b5c0589983 /llvm/tools/llvm-cov/SourceCoverageView.cpp | |
| parent | c80f7b866e769427045212e4968d284fd4546b9a (diff) | |
| download | llvm-d7d9e1069be675f40dd89e27143c0dc176ed498d.zip llvm-d7d9e1069be675f40dd89e27143c0dc176ed498d.tar.gz llvm-d7d9e1069be675f40dd89e27143c0dc176ed498d.tar.bz2 | |
Replace incorrect "#ifdef DEBUG" with "#ifndef NDEBUG".
The former is simply wrong -- the code will either never be used or will
always be used, rather than being dependent upon whether it's built with
debug assertions enabled.
The macro DEBUG isn't ever set by the llvm build system. But, the macro
DEBUG(X) is defined (unconditionally) if you happen to include
llvm/Support/Debug.h.
The code in Value.h which was erroneously protected by the #ifdef DEBUG
didn't even compile -- you can't cast<> from an LLVMOpaqueValue
directly. Fortunately, it was never invoked, as Core.cpp included
Value.h before Debug.h.
The conditionalized code in AArch64CollectLOH.cpp was previously always
used, as it includes Debug.h.
llvm-svn: 280056
Diffstat (limited to 'llvm/tools/llvm-cov/SourceCoverageView.cpp')
0 files changed, 0 insertions, 0 deletions