Boost C++ Libraries: Ticket #7099: boost/math/special_functions/detail/erf_inv.hpp", line 342: error: floating constant is out of range

anonymous — Fri, 06 Jul 2012 16:23:14 GMT

That's not really the correct fix in general, as it breaks the test to see whether the constant has underflowed to zero or not. I'll try and find another fix.

John Maddock — Sat, 07 Jul 2012 11:18:10 GMT

(In [79332]) Tentative fix for compilers that treat floating point constants smaller than a 64-bit double can hold as hard errors. Refs #7099.

John Maddock — Sat, 07 Jul 2012 11:19:31 GMT

Can you please try the change referenced above and let me know if it fixes the issue for you?

Many thanks.

1czajnik@… — Mon, 09 Jul 2012 08:49:55 GMT

Thanks for looking into this, John.

I've tried the change [79332]. Unfortunately it doesn't fix the problem.

Compiling a file as simple as:

#include <boost/math/special_functions/erf.hpp>

with the TI compiler results in error: "boost/math/special_functions/detail/erf_inv.hpp", lines 339-342,351,353: error: floating constant is out of range.

The errors on lines 351 and 353 look like a typo to me. On those two lines: s/BOOST_MATH_BIG_CONSTANT/BOOST_MATH_HUGE_CONSTANT

The errors on lines 339-342 seem to be reported even though this template is never instantiated. I checked this by adding BOOST_MPL_ASSERT(mpl::false_) to that function. The assertion is not reported by the compiler. Looks like the compiler reports an error when it sees a too large float constant at some early stage of compilation, even though the constant never gets used.

John Maddock — Thu, 12 Jul 2012 09:53:20 GMT

(In [79439]) Tentative fix #2 for TI compiler error. Refs #7099.

anonymous — Thu, 12 Jul 2012 09:54:41 GMT

OK, I've had a second try - this time using preprocessor rather than compile time logic. Can you try this again with Trunk? Thanks!

1czajnik@… — Thu, 12 Jul 2012 11:07:00 GMT

Yes, this fix [79439] works for my TI compiler, thanks again for looking into this.

Here's more info about preprocessor constants in this compiler, maybe they're of some use for a more concrete fix here:

__GNUC__=3 __GNUC_MINOR__=0
LDBL_MIN_EXP=-1021 LDBL_MAX_10_EXP=308
__TI_COMPILER_VERSION__=7003006

Cheers, Kris

John Maddock — Thu, 12 Jul 2012 11:15:07 GMT

Thanks for the info: the fix should be reasonably generic so I don't think we need a compiler specific fix here - hopefully!

status changed; resolution set

John Maddock — Thu, 12 Jul 2012 11:20:07 GMT

status new → closed
resolution → fixed

(In [79444]) Document #7099 as fixed. Fixes #7099.

milestone changed

John Maddock — Thu, 02 Aug 2012 12:00:46 GMT

milestone To Be Determined → Boost 1.52.0