John Maddock — Mon, 02 Apr 2018 17:19:00 GMT

Unfortunately, that configuration has dropped off our testing radar... can you tell me if the patch below fixes things?

 include/boost/multiprecision/float128.hpp | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/include/boost/multiprecision/float128.hpp b/include/boost/multiprecision/float128.hpp
index a729523..095634f 100644
--- a/include/boost/multiprecision/float128.hpp
+++ b/include/boost/multiprecision/float128.hpp
@@ -482,15 +482,16 @@ inline void eval_atan2(float128_backend& result, const float128_backend& a, cons
 {
    result.value() = atan2q(a.value(), b.value());
 }
+#ifndef BOOST_MP_USE_QUAD
 inline void eval_multiply_add(float128_backend& result, const float128_backend& a, const float128_backend& b, const float128_backend& c)
 {
    result.value() = fmaq(a.value(), b.value(), c.value());
 }
-
 inline int eval_signbit BOOST_PREVENT_MACRO_SUBSTITUTION(const float128_backend& arg)
 {
    return ::signbitq(arg.value());
 }
+#endif
 inline std::size_t hash_value(const float128_backend& val)
 {
@@ -550,6 +551,7 @@ inline std::size_t hash_value(const float128_backend& val)
       return log1pq(arg.backend().value());
    }
+#ifndef BOOST_MP_USE_QUAD
    template <multiprecision::expression_template_option ExpressionTemplates>
    inline boost::multiprecision::number<boost::multiprecision::backends::float128_backend, ExpressionTemplates> copysign BOOST_PREVENT_MACRO_SUBSTITUTION(const boost::multiprecision::number<boost::multiprecision::backends::float128_backend, ExpressionTemplates>& a, const boost::multiprecision::number<boost::multiprecision::backends::float128_backend, ExpressionTemplates>& b)
    {
@@ -564,6 +566,7 @@ inline std::size_t hash_value(const float128_backend& val)
    {
       result.value() = remquoq(a.value(), b.value(), pi);
    }
+#endif
 } // namespace multiprecision

Tue, 03 Apr 2018 11:27:01 GMT

Hi,

yes, I can confirm that the patch fixes the errors I was seeing. (side note, in the spirit of full transparency, I couldn't get git to apply the patch, almost sure it is because of copying from the web to a file and windows messing the line endings. The four inclusions were done by hand)

Thanks a lot.

Best,

status changed; resolution set

John Maddock — Tue, 03 Apr 2018 17:28:26 GMT

status new → closed
resolution → fixed

Thanks, fixed in develop.

Boost C++ Libraries: Ticket #13508: Errors compiling float128.h on windows with icc 17.0 and MSVC 14.11 toolchain

status changed; resolution set