summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAlp Toker <alp@nuanti.com>2013-12-09 00:12:56 +0000
committerAlp Toker <alp@nuanti.com>2013-12-09 00:12:56 +0000
commitcffb4869d4d255deae319a6e990dbbc2061aab90 (patch)
tree922a425afb27dd4a57d8b2170564f52eb530f63f
parent7c265cf46a03eeacd97155c131d69b863e4f3444 (diff)
downloadbcm5719-llvm-cffb4869d4d255deae319a6e990dbbc2061aab90.tar.gz
bcm5719-llvm-cffb4869d4d255deae319a6e990dbbc2061aab90.zip
Eliminate the last remaining header NDEBUG
VerifyDiagnosticConsumer is long-lived so the two additional members shouldn't have any impact on release builds. The clang headers are now free of NDEBUG conditionals. Let's keep it that way! Note that they're not yet structurally stable, pending a few fixes in the LLVM core headers. llvm-svn: 196739
-rw-r--r--clang/include/clang/Frontend/VerifyDiagnosticConsumer.h7
1 files changed, 1 insertions, 6 deletions
diff --git a/clang/include/clang/Frontend/VerifyDiagnosticConsumer.h b/clang/include/clang/Frontend/VerifyDiagnosticConsumer.h
index 95d7752517d..33d692a4020 100644
--- a/clang/include/clang/Frontend/VerifyDiagnosticConsumer.h
+++ b/clang/include/clang/Frontend/VerifyDiagnosticConsumer.h
@@ -217,24 +217,19 @@ private:
SrcManager = &SM;
}
-#ifndef NDEBUG
+ // These facilities are used for validation in debug builds.
class UnparsedFileStatus {
llvm::PointerIntPair<const FileEntry *, 1, bool> Data;
-
public:
UnparsedFileStatus(const FileEntry *File, bool FoundDirectives)
: Data(File, FoundDirectives) {}
-
const FileEntry *getFile() const { return Data.getPointer(); }
bool foundDirectives() const { return Data.getInt(); }
};
-
typedef llvm::DenseMap<FileID, const FileEntry *> ParsedFilesMap;
typedef llvm::DenseMap<FileID, UnparsedFileStatus> UnparsedFilesMap;
-
ParsedFilesMap ParsedFiles;
UnparsedFilesMap UnparsedFiles;
-#endif
public:
/// Create a new verifying diagnostic client, which will issue errors to
OpenPOWER on IntegriCloud