diff --git a/llvm/include/llvm/ADT/bit.h b/llvm/include/llvm/ADT/bit.h --- a/llvm/include/llvm/ADT/bit.h +++ b/llvm/include/llvm/ADT/bit.h @@ -22,23 +22,28 @@ // This implementation of bit_cast is different from the C++17 one in two ways: // - It isn't constexpr because that requires compiler support. // - It requires trivially-constructible To, to avoid UB in the implementation. -template ::type +template < + typename To, typename From, + typename = std::enable_if_t #if (__has_feature(is_trivially_constructible) && defined(_LIBCPP_VERSION)) || \ (defined(__GNUC__) && __GNUC__ >= 5) - , typename = typename std::is_trivially_constructible::type + , + typename = std::enable_if_t::value> #elif __has_feature(is_trivially_constructible) - , typename = typename std::enable_if<__is_trivially_constructible(To)>::type + , + typename = std::enable_if_t<__is_trivially_constructible(To)> #else // See comment below. #endif #if (__has_feature(is_trivially_copyable) && defined(_LIBCPP_VERSION)) || \ (defined(__GNUC__) && __GNUC__ >= 5) - , typename = typename std::enable_if::value>::type - , typename = typename std::enable_if::value>::type + , + typename = std::enable_if_t::value>, + typename = std::enable_if_t::value> #elif __has_feature(is_trivially_copyable) - , typename = typename std::enable_if<__is_trivially_copyable(To)>::type - , typename = typename std::enable_if<__is_trivially_copyable(From)>::type + , + typename = std::enable_if_t<__is_trivially_copyable(To)>, + typename = std::enable_if_t<__is_trivially_copyable(From)> #else // This case is GCC 4.x. clang with libc++ or libstdc++ never get here. Unlike // llvm/Support/type_traits.h's is_trivially_copyable we don't want to @@ -46,7 +51,7 @@ // compilation failures on the bots instead of locally. That's acceptable // because it's very few developers, and only until we move past C++11. #endif -> + > inline To bit_cast(const From &from) noexcept { To to; std::memcpy(&to, &from, sizeof(To));