Merge pull request #329 from cppalliance/develop

Merge to master for v1.6.1

Author: mborland

include/boost/int128/detail/int128_imp.hpp

@@ -24,10 +24,8 @@ namespace boost {
 namespace int128 {
 
 struct
-    #if defined(BOOST_INT128_HAS_INT128) || defined(BOOST_INT128_HAS_MSVC_INT128)
+    #if (defined(BOOST_INT128_HAS_INT128) || defined(BOOST_INT128_HAS_MSVC_INT128)) && !defined(_M_IX86)
     alignas(alignof(detail::builtin_i128))
-    #else
-    alignas(16)
     #endif
 int128_t
 {

include/boost/int128/detail/uint128_imp.hpp

@@ -25,10 +25,8 @@ namespace boost {
 namespace int128 {
 
 BOOST_INT128_EXPORT struct
-    #if defined(BOOST_INT128_HAS_INT128) || defined(BOOST_INT128_HAS_MSVC_INT128)
+    #if (defined(BOOST_INT128_HAS_INT128) || defined(BOOST_INT128_HAS_MSVC_INT128)) && !defined(_M_IX86)
     alignas(alignof(detail::builtin_u128))
-    #else
-    alignas(16)
     #endif
 uint128_t
 {
@@ -401,8 +399,8 @@ BOOST_INT128_EXPORT constexpr bool operator==(const uint128_t lhs, const uint128
     }
     else
     {
-        __m128i a = _mm_load_si128(reinterpret_cast<const __m128i*>(&lhs));
-        __m128i b = _mm_load_si128(reinterpret_cast<const __m128i*>(&rhs));
+        __m128i a = _mm_loadu_si128(reinterpret_cast<const __m128i*>(&lhs));
+        __m128i b = _mm_loadu_si128(reinterpret_cast<const __m128i*>(&rhs));
         __m128i cmp = _mm_cmpeq_epi32(a, b);
 
         return _mm_movemask_ps(_mm_castsi128_ps(cmp)) == 0xF;
@@ -537,8 +535,8 @@ BOOST_INT128_EXPORT constexpr bool operator!=(const uint128_t lhs, const uint128
     }
     else
     {
-        __m128i a = _mm_load_si128(reinterpret_cast<const __m128i*>(&lhs));
-        __m128i b = _mm_load_si128(reinterpret_cast<const __m128i*>(&rhs));
+        __m128i a = _mm_loadu_si128(reinterpret_cast<const __m128i*>(&lhs));
+        __m128i b = _mm_loadu_si128(reinterpret_cast<const __m128i*>(&rhs));
         __m128i cmp = _mm_cmpeq_epi32(a, b);
 
         return _mm_movemask_ps(_mm_castsi128_ps(cmp)) != 0xF;

include/boost/int128/string.hpp

@@ -20,9 +20,9 @@ namespace int128 {
 template <typename T>
 auto to_string(const T& value) -> std::enable_if_t<(std::is_same<T, int128_t>::value || std::is_same<T, uint128_t>::value), std::string>
 {
-    char buffer[64U] {};
+    char buffer[64];
     const auto last {detail::mini_to_chars(buffer, value, 10, false)};
-    return std::string{last};
+    return std::string{last, buffer + sizeof(buffer)};
 }
 
 } // namespace int128