Require that SIMD types are trivially copyable

hrabalik · Jan 22, 2016 · 68f3262 · 68f3262
1 parent 2523662
commit 68f3262
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 1 deletion.
diff --git a/include/simdify/impl/named_array.hpp b/include/simdify/impl/named_array.hpp
@@ -271,7 +271,7 @@ namespace simd {
         }
 
         SIMDIFY_INL void swap(named_array& rhs) {
-            if (std::is_trivial<T>::value) {
+            if (std::is_trivially_copyable<T>::value) {
                 std::swap(*this, rhs);
             }
             else {

diff --git a/test/avx.cpp b/test/avx.cpp
@@ -63,6 +63,9 @@ TEST_CASE("AVX basic guarantees", "[simd_t][x86][avx]") {
     REQUIRE(sizeof(F::e_t) * F::W == sizeof(F));
     REQUIRE(sizeof(U::e_t) * U::W == sizeof(U));
     REQUIRE(sizeof(S::e_t) * S::W == sizeof(S));
+    REQUIRE((std::is_trivially_copyable<F>::value));
+    REQUIRE((std::is_trivially_copyable<U>::value));
+    REQUIRE((std::is_trivially_copyable<S>::value));
 }
 
 TEST_CASE("AVX explicit construction", "[simd_t][x86][avx]") {

diff --git a/test/dum.cpp b/test/dum.cpp
@@ -57,6 +57,9 @@ TEST_CASE("DUM basic guarantees", "[simd_t][x86][dum]") {
     REQUIRE(sizeof(F::e_t) * F::W == sizeof(F));
     REQUIRE(sizeof(U::e_t) * U::W == sizeof(U));
     REQUIRE(sizeof(S::e_t) * S::W == sizeof(S));
+    REQUIRE((std::is_trivially_copyable<F>::value));
+    REQUIRE((std::is_trivially_copyable<U>::value));
+    REQUIRE((std::is_trivially_copyable<S>::value));
 }
 
 TEST_CASE("DUM explicit construction", "[simd_t][x86][dum]") {

diff --git a/test/sse.cpp b/test/sse.cpp
@@ -57,6 +57,9 @@ TEST_CASE("SSE basic guarantees", "[simd_t][x86][sse]") {
     REQUIRE(sizeof(F::e_t) * F::W == sizeof(F));
     REQUIRE(sizeof(U::e_t) * U::W == sizeof(U));
     REQUIRE(sizeof(S::e_t) * S::W == sizeof(S));
+    REQUIRE((std::is_trivially_copyable<F>::value));
+    REQUIRE((std::is_trivially_copyable<U>::value));
+    REQUIRE((std::is_trivially_copyable<S>::value));
 }
 
 TEST_CASE("SSE explicit construction", "[simd_t][x86][sse]") {