Moving to static_assert now that c++11 is available.
authorTim King <taking@cs.nyu.edu>
Mon, 17 Jul 2017 05:13:10 +0000 (22:13 -0700)
committerGitHub <noreply@github.com>
Mon, 17 Jul 2017 05:13:10 +0000 (22:13 -0700)
examples/hashsmt/sha1.hpp
src/context/cdinsert_hashmap.h
src/context/cdlist.h
src/context/cdtrail_hashmap.h
src/util/Makefile.am
src/util/index.cpp [new file with mode: 0644]
src/util/index.h

index 4bfee50fc57b19b4ec390c71c4a0ab918843886b..72c560dff703cb5ce1ccffe59acfc8b120b138e6 100644 (file)
@@ -30,9 +30,9 @@
 // Note: this implementation does not handle message longer than
 //       2^32 bytes.
 
-#pragma once 
+#ifndef __CVC4__EXAMPLES__HASHSMT__SHA1_H
+#define __CVC4__EXAMPLES__HASHSMT__SHA1_H
 
-#include <boost/static_assert.hpp>
 #include <cstddef>
 
 #include "word.h"
@@ -45,8 +45,10 @@ namespace std {
 
 namespace hashsmt {
 
-BOOST_STATIC_ASSERT(sizeof(unsigned char)*8 == 8);
-BOOST_STATIC_ASSERT(sizeof(unsigned int)*8 == 32);
+static_assert(sizeof(unsigned char)*8 == 8,
+              "Unexpected size for unsigned char");
+static_assert(sizeof(unsigned int)*8 == 32,
+              "Unexpected size for unsigned int");
 
 inline cvc4_uint32 left_rotate(cvc4_uint32 x, std::size_t n)
 {
@@ -224,3 +226,4 @@ inline void sha1::get_digest(digest_type digest)
 
 } // namespace hashsmt
 
+#endif /* __CVC4__EXAMPLES__HASHSMT__SHA1_H */
index f53d60cf99257cc4cc268a06c0e204e0feb00b83..ef8f661fa7f435c41c6cdc0b2ec26210f3cd398d 100644 (file)
@@ -33,7 +33,6 @@
 
 #include "cvc4_private.h"
 
-#include <boost/static_assert.hpp>
 #include <deque>
 #include <ext/hash_map>
 #include <utility>
@@ -383,9 +382,8 @@ public:
   }
 };/* class CDInsertHashMap<> */
 
-
 template <class Data, class HashFcn>
-class CDInsertHashMap <TNode, Data, HashFcn > : public ContextObj {
+class CDInsertHashMap<TNode, Data, HashFcn> : public ContextObj {
   /* CDInsertHashMap is challenging to get working with TNode.
    * Consider using CDHashMap<TNode,...> instead.
    *
@@ -397,7 +395,8 @@ class CDInsertHashMap <TNode, Data, HashFcn > : public ContextObj {
    * hashed. Getting the order right with a guarantee is too hard.
    */
 
-  BOOST_STATIC_ASSERT(sizeof(Data) == 0);
+  static_assert(sizeof(Data) == 0,
+                "Cannot create a CDInsertHashMap with TNode keys");
 };
 
 }/* CVC4::context namespace */
index 7cee41b91e9034ae7be175d7404253e02323de73..3dab675c3f456438d2123346b44e7f5c3d068d71 100644 (file)
@@ -20,7 +20,6 @@
 #ifndef __CVC4__CONTEXT__CDLIST_H
 #define __CVC4__CONTEXT__CDLIST_H
 
-#include <boost/static_assert.hpp>
 #include <iterator>
 #include <memory>
 #include <string>
@@ -417,21 +416,21 @@ public:
   }
 };/* class CDList<> */
 
-
 template <class T, class CleanUp>
-class CDList <T, CleanUp, ContextMemoryAllocator<T> > : public ContextObj {
+class CDList<T, CleanUp, ContextMemoryAllocator<T> > : public ContextObj {
   /* CDList is incompatible for use with a ContextMemoryAllocator.
    * Consider using CDChunkList<T> instead.
    *
    * Explanation:
-   * If ContextMemoryAllocator is used and d_list grows at a deeper context level
-   * the reallocated will be reallocated in a context memory region that can be
-   * destroyed on pop. To support this, a full copy of d_list would have to be made.
-   * As this is unacceptable for performance in other situations, we do not do
-   * this.
+   * If ContextMemoryAllocator is used and d_list grows at a deeper context
+   * level the reallocated will be reallocated in a context memory region that
+   * can be destroyed on pop. To support this, a full copy of d_list would have
+   * to be made. As this is unacceptable for performance in other situations, we
+   * do not do this.
    */
 
-  BOOST_STATIC_ASSERT(sizeof(T) == 0);
+  static_assert(sizeof(T) == 0,
+                "Cannot create a CDList with a ContextMemoryAllocator.");
 };
 
 }/* CVC4::context namespace */
index 9f364eac5f811073eedaff91db1927f8a7212cd8..0d265271d9ec30df63b2962f748a2ad94899d3a6 100644 (file)
@@ -44,7 +44,6 @@
 
 #pragma once
 
-#include <boost/static_assert.hpp>
 #include <ext/hash_map>
 #include <deque>
 #include <utility>
@@ -551,7 +550,7 @@ public:
 };/* class CDTrailHashMap<> */
 
 template <class Data, class HashFcn>
-class CDTrailHashMap <TNode, Data, HashFcn > : public ContextObj {
+class CDTrailHashMap<TNode, Data, HashFcn> : public ContextObj {
   /* CDTrailHashMap is challenging to get working with TNode.
    * Consider using CDHashMap<TNode,...> instead.
    *
@@ -563,7 +562,8 @@ class CDTrailHashMap <TNode, Data, HashFcn > : public ContextObj {
    * hashed. Getting the order right with a guarantee is too hard.
    */
 
-  BOOST_STATIC_ASSERT(sizeof(Data) == 0);
+  static_assert(sizeof(Data) == 0,
+                "Cannot create a CDTrailHashMap with TNode keys");
 };
 
 }/* CVC4::context namespace */
index cd6f0e11c45b6393874ad66ed36d52376e49688e..877525de7d9f52a33e33aee7c26877b76d927816 100644 (file)
@@ -34,6 +34,7 @@ libutil_la_SOURCES = \
        floatingpoint.h \
        gmp_util.h \
        hash.h \
+       index.cpp \
        index.h \
        maybe.h \
        ntuple.h \
diff --git a/src/util/index.cpp b/src/util/index.cpp
new file mode 100644 (file)
index 0000000..bd15e2d
--- /dev/null
@@ -0,0 +1,21 @@
+#include "util/index.h"
+
+#include <cstddef>
+#include <limits>
+
+namespace CVC4 {
+
+static_assert(sizeof(Index) <= sizeof(size_t),
+              "Index cannot be larger than size_t");
+static_assert(!std::numeric_limits<Index>::is_signed,
+              "Index must be unsigned");
+
+/* Discussion: Why is Index a uint32_t instead of size_t (or uint_fast32_t)?
+ *
+ * size_t is a more appropriate choice than uint32_t as the choice is dictated
+ * by uniqueness in arrays and vectors. These correspond to size_t. However, the
+ * using size_t with a sizeof == 8 on 64 bit platforms is noticeably slower.
+ * (Limited testing suggests a ~1/16 of running time.) Interestingly,
+ * uint_fast32_t also has a sizeof == 8 on x86_64.
+ */
+}/* CVC4 namespace */
index a8910a696db85e4395ada8400d2ff9d3a22774ea..7329e3f8d86367844ec9f8f75f0656656a7ae372 100644 (file)
@@ -9,39 +9,23 @@
  ** All rights reserved.  See the file COPYING in the top-level source
  ** directory for licensing information.\endverbatim
  **
- ** \brief [[ Add one-line brief description here ]]
+ ** \brief Standardized type for efficient array indexing.
  **
- ** [[ Add lengthier description here ]]
- ** \todo document this file
+ ** Standardized type for efficient array indexing.
  **/
 
 #include "cvc4_private.h"
 
-#pragma once
+#ifndef __CVC4__INDEX_H
+#define __CVC4__INDEX_H
 
-#include <stdint.h>
-#include <boost/static_assert.hpp>
-#include <limits>
+#include <cstdint>
 
 namespace CVC4 {
 
-/**
- * Index is an unsigned integer used for array indexing.
- *
- * This gives a standardized type for independent pieces of code to use as an agreement.
- */
-typedef uint32_t Index;
-
-BOOST_STATIC_ASSERT(sizeof(Index) <= sizeof(size_t));
-BOOST_STATIC_ASSERT(!std::numeric_limits<Index>::is_signed);
-
-/* Discussion: Why is Index a uint32_t instead of size_t (or uint_fast32_t)?
- *
- * size_t is a more appropriate choice than uint32_t as the choice is dictated by
- * uniqueness in arrays and vectors. These correspond to size_t.
- * However, the using size_t with a sizeof == 8 on 64 bit platforms is noticeably
- * slower. (Limited testing suggests a ~1/16 of running time.)
- * (Interestingly, uint_fast32_t also has a sizeof == 8 on x86_64. Filthy Liars!)
- */
+/** Index is a standardized unsigned integer used for efficient indexing. */
+using Index = uint32_t;
 
 }/* CVC4 namespace */
+
+#endif /* __CVC4__INDEX_H */