The Windows build is just too weird; there's no real cost to doing the concurrency...

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@97840 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/include/clang/Frontend/ASTUnit.h b/include/clang/Frontend/ASTUnit.h
index 70a1b2bb16255d84a5f51ac0b32d8ae5c68a73d0..7f11c85ae35b1eed1e4132a876ae5b5bbda8c78b 100644 (file)
--- a/include/clang/Frontend/ASTUnit.h
+++ b/include/clang/Frontend/ASTUnit.h
@@ -89,7 +89,6 @@ class ASTUnit {
   /// destroyed.
   llvm::SmallVector<llvm::sys::Path, 4> TemporaryFiles;
 
-#ifdef _DEBUG
   /// \brief Simple hack to allow us to assert that ASTUnit is not being
   /// used concurrently, which is not supported.
   ///
@@ -99,35 +98,26 @@ class ASTUnit {
   unsigned int ConcurrencyCheckValue;
   static const unsigned int CheckLocked = 28573289;
   static const unsigned int CheckUnlocked = 9803453;
-#endif
   
   ASTUnit(const ASTUnit&); // DO NOT IMPLEMENT
   ASTUnit &operator=(const ASTUnit &); // DO NOT IMPLEMENT
   
 public:
   class ConcurrencyCheck {
-#ifdef _DEBUG
     volatile ASTUnit &Self;
-#endif
     
   public:
     explicit ConcurrencyCheck(ASTUnit &Self)
-#ifdef _DEBUG
       : Self(Self) 
-#endif
     { 
-#ifdef _DEBUG
       assert(Self.ConcurrencyCheckValue == CheckUnlocked && 
              "Concurrent access to ASTUnit!");
       Self.ConcurrencyCheckValue = CheckLocked;
-#endif
     }
     
-#ifdef _DEBUG
     ~ConcurrencyCheck() {
       Self.ConcurrencyCheckValue = CheckUnlocked;
     }
-#endif
   };
   friend class ConcurrencyCheck;
   

diff --git a/lib/Frontend/ASTUnit.cpp b/lib/Frontend/ASTUnit.cpp
index 3bf5f796c43e53df10b9590157e9b1a31413fd24..63cf98d3da5d6fdbddd2b2f1bd274cc47fb1ede0 100644 (file)
--- a/lib/Frontend/ASTUnit.cpp
+++ b/lib/Frontend/ASTUnit.cpp
@@ -39,9 +39,7 @@ ASTUnit::ASTUnit(bool _MainFileIsAST)
   : MainFileIsAST(_MainFileIsAST), ConcurrencyCheckValue(CheckUnlocked) {
 }
 ASTUnit::~ASTUnit() {
-#ifdef _DEBUG
   ConcurrencyCheckValue = CheckLocked;
-#endif
   for (unsigned I = 0, N = TemporaryFiles.size(); I != N; ++I)
     TemporaryFiles[I].eraseFromDisk();
 }