]> granicus.if.org Git - llvm/commitdiff
Remove the is_trivially_copyable check entirely.
authorZachary Turner <zturner@google.com>
Sat, 18 Feb 2017 01:51:00 +0000 (01:51 +0000)
committerZachary Turner <zturner@google.com>
Sat, 18 Feb 2017 01:51:00 +0000 (01:51 +0000)
This is still breaking builds because some compilers think
this type is not trivially copyable even when it should be.

Reverting this static_assert until I have time to investigate.

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

include/llvm/DebugInfo/MSF/StreamReader.h
include/llvm/DebugInfo/MSF/StreamWriter.h

index 719a26def087b21240626cc65871365cf03288e1..93131c6eb65a361a4ad29f4e8d11ec3d56c16571 100644 (file)
@@ -65,8 +65,6 @@ public:
   }
 
   template <typename T> Error readObject(const T *&Dest) {
-    static_assert(isPodLike<T>::value,
-                  "Can only read trivially copyable object types!");
     ArrayRef<uint8_t> Buffer;
     if (auto EC = readBytes(Buffer, sizeof(T)))
       return EC;
@@ -76,8 +74,6 @@ public:
 
   template <typename T>
   Error readArray(ArrayRef<T> &Array, uint32_t NumElements) {
-    static_assert(isPodLike<T>::value,
-                  "Can only read trivially copyable object types!");
     ArrayRef<uint8_t> Bytes;
     if (NumElements == 0) {
       Array = ArrayRef<T>();
@@ -104,8 +100,6 @@ public:
 
   template <typename T>
   Error readArray(FixedStreamArray<T> &Array, uint32_t NumItems) {
-    static_assert(isPodLike<T>::value,
-                  "Can only read trivially copyable object types!");
     if (NumItems == 0) {
       Array = FixedStreamArray<T>();
       return Error::success();
index 69bb5505efba480b53d2b7821a7d555407419b16..9444b92ad2be8d8e92933b6d891f27ec7f880518 100644 (file)
@@ -17,7 +17,6 @@
 #include "llvm/DebugInfo/MSF/StreamRef.h"
 #include "llvm/Support/Endian.h"
 #include "llvm/Support/Error.h"
-#include "llvm/Support/type_traits.h"
 #include <cstdint>
 #include <type_traits>
 
@@ -62,15 +61,11 @@ public:
                   "writeObject should not be used with pointers, to write "
                   "the pointed-to value dereference the pointer before calling "
                   "writeObject");
-    static_assert(isPodLike<T>::value,
-                  "Can only serialize trivially copyable object types");
     return writeBytes(
         ArrayRef<uint8_t>(reinterpret_cast<const uint8_t *>(&Obj), sizeof(T)));
   }
 
   template <typename T> Error writeArray(ArrayRef<T> Array) {
-    static_assert(isPodLike<T>::value,
-                  "Can only serialize trivially copyable object types");
     if (Array.empty())
       return Error::success();
 
@@ -88,8 +83,6 @@ public:
   }
 
   template <typename T> Error writeArray(FixedStreamArray<T> Array) {
-    static_assert(isPodLike<T>::value,
-                  "Can only serialize trivially copyable object types");
     return writeStreamRef(Array.getUnderlyingStream());
   }