Minor cleanups for deleted functions to ensure they're a bit more robustly "deleted".

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

diff --git a/include/clang/Analysis/Analyses/ThreadSafetyUtil.h b/include/clang/Analysis/Analyses/ThreadSafetyUtil.h
index a36195f491409336b24a44d75a0c29691b08a36c..c5cd8fc63990b911f8e17ca7a9650f22820a0b14 100644 (file)
--- a/include/clang/Analysis/Analyses/ThreadSafetyUtil.h
+++ b/include/clang/Analysis/Analyses/ThreadSafetyUtil.h
@@ -139,7 +139,7 @@ public:
   }
 
 private:
-  SimpleArray(const SimpleArray<T> &A) { }
+  SimpleArray(const SimpleArray<T> &A) LLVM_DELETED_FUNCTION;
 
   T *Data;
   size_t Size;
@@ -157,7 +157,6 @@ private:
 // The init(), destroy(), and makeWritable() methods will change state.
 template<typename T>
 class CopyOnWriteVector {
-private:
   class VectorData {
   public:
     VectorData() : NumRefs(1) { }
@@ -167,9 +166,10 @@ private:
     std::vector<T> Vect;
   };
 
+  CopyOnWriteVector(const CopyOnWriteVector &V) LLVM_DELETED_FUNCTION;
+
 public:
   CopyOnWriteVector() : Data(0) {}
-  CopyOnWriteVector(const CopyOnWriteVector &V) LLVM_DELETED_FUNCTION;
   CopyOnWriteVector(CopyOnWriteVector &&V) : Data(V.Data) { V.Data = 0; }
   ~CopyOnWriteVector() { destroy(); }