Skip to content

Commit

Permalink
[libc][NFC] Add library of floating point test matchers.
Browse files Browse the repository at this point in the history
This eliminates UnitTest's dependency on FPUtil and hence prevents
non-math tests from depending indirectly on FPUtil. The patch
essentially moves some of the existing pieces into a library of its own.

Along the way, renamed add_math_unittest to add_fp_unittest.

Reviewed By: lntue

Differential Revision: https://reviews.llvm.org/D85486
  • Loading branch information
Siva Chandra Reddy committed Aug 8, 2020
1 parent 5c9c4ad commit db936e0
Show file tree
Hide file tree
Showing 12 changed files with 375 additions and 239 deletions.
83 changes: 42 additions & 41 deletions libc/test/src/math/CMakeLists.txt
@@ -1,6 +1,6 @@
add_libc_testsuite(libc_math_unittests)

function(add_math_unittest name)
function(add_fp_unittest name)
cmake_parse_arguments(
"MATH_UNITTEST"
"NEED_MPFR" # No optional arguments
Expand All @@ -17,13 +17,14 @@ function(add_math_unittest name)
endif()

add_libc_unittest(${name} ${MATH_UNITTEST_UNPARSED_ARGUMENTS})
get_fq_target_name(${name} fq_target_name)
target_link_libraries(${fq_target_name} PRIVATE LibcFPTestHelpers)
if(MATH_UNITTEST_NEED_MPFR)
get_fq_target_name(${name} fq_target_name)
target_link_libraries(${fq_target_name} PRIVATE libcMPFRWrapper -lmpfr -lgmp)
endif()
endfunction(add_math_unittest)
endfunction(add_fp_unittest)

add_math_unittest(
add_fp_unittest(
cosf_test
NEED_MPFR
SUITE
Expand All @@ -39,7 +40,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
sinf_test
NEED_MPFR
SUITE
Expand All @@ -55,7 +56,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
sincosf_test
NEED_MPFR
SUITE
Expand All @@ -71,7 +72,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
fabs_test
NEED_MPFR
SUITE
Expand All @@ -84,7 +85,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
fabsf_test
NEED_MPFR
SUITE
Expand All @@ -97,7 +98,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
fabsl_test
NEED_MPFR
SUITE
Expand All @@ -110,7 +111,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
trunc_test
NEED_MPFR
SUITE
Expand All @@ -123,7 +124,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
truncf_test
NEED_MPFR
SUITE
Expand All @@ -136,7 +137,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
truncl_test
NEED_MPFR
SUITE
Expand All @@ -149,7 +150,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
ceil_test
NEED_MPFR
SUITE
Expand All @@ -162,7 +163,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
ceilf_test
NEED_MPFR
SUITE
Expand All @@ -175,7 +176,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
ceill_test
NEED_MPFR
SUITE
Expand All @@ -188,7 +189,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
floor_test
NEED_MPFR
SUITE
Expand All @@ -201,7 +202,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
floorf_test
NEED_MPFR
SUITE
Expand All @@ -214,7 +215,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
floorl_test
NEED_MPFR
SUITE
Expand All @@ -227,7 +228,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
round_test
NEED_MPFR
SUITE
Expand All @@ -240,7 +241,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
roundf_test
NEED_MPFR
SUITE
Expand All @@ -253,7 +254,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
roundl_test
NEED_MPFR
SUITE
Expand All @@ -266,7 +267,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
expf_test
NEED_MPFR
SUITE
Expand All @@ -280,7 +281,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
exp2f_test
NEED_MPFR
SUITE
Expand All @@ -294,7 +295,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
copysign_test
SUITE
libc_math_unittests
Expand All @@ -306,7 +307,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
copysignf_test
SUITE
libc_math_unittests
Expand All @@ -318,7 +319,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
copysignl_test
SUITE
libc_math_unittests
Expand All @@ -330,7 +331,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
frexp_test
SUITE
libc_math_unittests
Expand All @@ -342,7 +343,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
frexpf_test
SUITE
libc_math_unittests
Expand All @@ -354,7 +355,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
frexpl_test
SUITE
libc_math_unittests
Expand All @@ -366,7 +367,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
logb_test
SUITE
libc_math_unittests
Expand All @@ -378,7 +379,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
logbf_test
SUITE
libc_math_unittests
Expand All @@ -390,7 +391,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
logbl_test
SUITE
libc_math_unittests
Expand All @@ -402,7 +403,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
modf_test
SUITE
libc_math_unittests
Expand All @@ -414,7 +415,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
modff_test
SUITE
libc_math_unittests
Expand All @@ -426,7 +427,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
modfl_test
SUITE
libc_math_unittests
Expand All @@ -438,7 +439,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
fminf_test
SUITE
libc_math_unittests
Expand All @@ -450,7 +451,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
fmin_test
SUITE
libc_math_unittests
Expand All @@ -462,7 +463,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
fminl_test
SUITE
libc_math_unittests
Expand All @@ -474,7 +475,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
fmaxf_test
SUITE
libc_math_unittests
Expand All @@ -486,7 +487,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
fmax_test
SUITE
libc_math_unittests
Expand All @@ -498,7 +499,7 @@ add_math_unittest(
libc.utils.FPUtil.fputil
)

add_math_unittest(
add_fp_unittest(
fmaxl_test
SUITE
libc_math_unittests
Expand Down

0 comments on commit db936e0

Please sign in to comment.