Skip to content

Commit 5daedec

Browse files
committed
Coverage: many tests were not being run due to (accidental) dependency.
Remove dependency on 128 bit std/boost::floatmax_t except where really needed.
1 parent 490af2e commit 5daedec

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

test/Jamfile.v2

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -101,10 +101,14 @@ local float128_type_intel_quad =
101101
local float128_type_gcc =
102102
[ check-target-builds ../config//has_float128 "GCC libquadmath and __float128 support"
103103
: <toolset>gcc:<library>quadmath <toolset>gcc:<define>BOOST_MATH_TEST_FLOAT128 ] ;
104-
local float128_type_floatmax =
104+
local float128_type_floatmax = $(float128_type_gcc) $(float128_type_intel_quad)
105105
[ check-target-builds ../config//has_128bit_floatmax_t "128-bit floatmax_t" : : <build>no ] ;
106+
#
107+
# DO NOT include $(float128_type_floatmax) in this macro, otherwise the whole test
108+
# gets disabled when std::float128_t is not available (which is most of the time!!)
109+
#
106110
local float128_type =
107-
$(float128_type_intel_quad) $(float128_type_gcc) $(float128_type_floatmax) ;
111+
$(float128_type_intel_quad) $(float128_type_gcc) ;
108112

109113
test-suite special_fun :
110114
[ run test_1F0.cpp /boost/test//boost_unit_test_framework : : : [ requires cxx11_auto_declarations cxx11_lambdas cxx11_unified_initialization_syntax cxx11_smart_ptr ] <define>TEST=1 : test_1F0_1 ] # hypergeometric_pFq_checked_series.hpp uses auto, the rest are from quadrature tests.
@@ -1380,7 +1384,7 @@ rule get_float128_tests
13801384
: # command line
13811385
: # input files
13821386
: # requirements
1383-
$(float128_type)
1387+
$(float128_type_floatmax)
13841388
<define>BOOST_ALL_NO_LIB
13851389
: $(source:B)_floatmax_t ] ;
13861390
}

0 commit comments

Comments
 (0)