diff options
| author | Alp Toker <alp@nuanti.com> | 2013-12-09 00:12:56 +0000 |
|---|---|---|
| committer | Alp Toker <alp@nuanti.com> | 2013-12-09 00:12:56 +0000 |
| commit | cffb4869d4d255deae319a6e990dbbc2061aab90 (patch) | |
| tree | 922a425afb27dd4a57d8b2170564f52eb530f63f | |
| parent | 7c265cf46a03eeacd97155c131d69b863e4f3444 (diff) | |
| download | bcm5719-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.h | 7 |
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 |

