diff options
author | Rasmus Munk Larsen <rmlarsen@google.com> | 2020-10-07 19:05:18 +0000 |
---|---|---|
committer | Rasmus Munk Larsen <rmlarsen@google.com> | 2020-10-07 19:05:18 +0000 |
commit | b43102440489df9d0175c88e602dfa425b574a94 (patch) | |
tree | 9325c3401de7047451d4a59ad343cdf1c5a83679 /test | |
parent | f66f3393e3d567e5c8b138fbad69b316214a4ce9 (diff) |
Don't make assumptions about NaN-propagation for pmin/pmax - it various across platforms.
Change test to only test for NaN-propagation for pfmin/pfmax.
Diffstat (limited to 'test')
-rw-r--r-- | test/packetmath.cpp | 20 |
1 files changed, 17 insertions, 3 deletions
diff --git a/test/packetmath.cpp b/test/packetmath.cpp index dd3e5b41e..6cde7e87b 100644 --- a/test/packetmath.cpp +++ b/test/packetmath.cpp @@ -763,6 +763,20 @@ void packetmath_real<bfloat16, typename internal::packet_traits<bfloat16>::type> } +template <typename Scalar> +Scalar propagate_nan_max(const Scalar& a, const Scalar& b) { + if ((std::isnan)(a)) return a; + if ((std::isnan)(b)) return b; + return (std::max)(a,b); +} + +template <typename Scalar> +Scalar propagate_nan_min(const Scalar& a, const Scalar& b) { + if ((std::isnan)(a)) return a; + if ((std::isnan)(b)) return b; + return (std::min)(a,b); +} + template <typename Scalar, typename Packet> void packetmath_notcomplex() { typedef internal::packet_traits<Scalar> PacketTraits; @@ -829,12 +843,12 @@ void packetmath_notcomplex() { data1[i] = internal::random<bool>() ? std::numeric_limits<Scalar>::quiet_NaN() : Scalar(0); data1[i + PacketSize] = internal::random<bool>() ? std::numeric_limits<Scalar>::quiet_NaN() : Scalar(0); } - // Test NaN propagation for pmin and pmax. It should be equivalent to std::min. - CHECK_CWISE2_IF(PacketTraits::HasMin, (std::min), internal::pmin); - CHECK_CWISE2_IF(PacketTraits::HasMax, (std::max), internal::pmax); // Test NaN propagation for pfmin and pfmax. It should be equivalent to std::fmin. + // Note: NaN propagation is implementation defined for pmin/pmax, so we do not test it here. CHECK_CWISE2_IF(PacketTraits::HasMin, fmin, internal::pfmin); CHECK_CWISE2_IF(PacketTraits::HasMax, fmax, internal::pfmax); + CHECK_CWISE2_IF(PacketTraits::HasMin, propagate_nan_min, internal::pfmin_nan); + CHECK_CWISE2_IF(PacketTraits::HasMax, propagate_nan_max, internal::pfmax_nan); } template <> |